Skip to content

improvement(spark): Record the partition choosing sort time for analysis#2364

Merged
zuston merged 1 commit intoapache:masterfrom
zuston:sortTime
Feb 7, 2025
Merged

improvement(spark): Record the partition choosing sort time for analysis#2364
zuston merged 1 commit intoapache:masterfrom
zuston:sortTime

Conversation

@zuston
Copy link
Member

@zuston zuston commented Feb 7, 2025

What changes were proposed in this pull request?

Record the partition choosing sort time for analysis

Why are the changes needed?

For better optimize write duration

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Needn't

@zuston
Copy link
Member Author

zuston commented Feb 7, 2025

cc @jerqi

@github-actions
Copy link

github-actions bot commented Feb 7, 2025

Test Results

 2 981 files  ±0   2 981 suites  ±0   6h 28m 24s ⏱️ -38s
 1 101 tests ±0   1 098 ✅  - 1   2 💤 ±0  0 ❌ ±0  1 🔥 +1 
13 789 runs  ±0  13 757 ✅  - 2  30 💤 ±0  0 ❌ ±0  2 🔥 +2 

For more details on these errors, see this check.

Results for commit 0261510. ± Comparison against base commit 9e5cd61.

@zuston zuston merged commit e5cfc4a into apache:master Feb 7, 2025
38 of 43 checks passed
zuston added a commit to zuston/incubator-uniffle that referenced this pull request Feb 11, 2025
…sis (apache#2364)

### What changes were proposed in this pull request?

Record the partition choosing sort time for analysis

### Why are the changes needed?

For better optimize write duration

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

Needn't

Co-authored-by: Junfan Zhang <zhangjunfan@qiyi.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants