-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[Refactor](shuffle) Fix wrong deal of batching block in TabletSinkHashPartitioner #48720
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
|
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
|
run buildall |
05c9422 to
5a86da5
Compare
|
run buildall |
TPC-H: Total hot run time: 32869 ms |
TPC-DS: Total hot run time: 192644 ms |
ClickBench: Total hot run time: 31.03 s |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
yiguolei
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These files should add BE UT
|
run buildall |
TPC-H: Total hot run time: 32475 ms |
TPC-DS: Total hot run time: 192865 ms |
ClickBench: Total hot run time: 30.96 s |
a1aa63a to
e74e9c5
Compare
|
run buildall |
TPC-H: Total hot run time: 33882 ms |
TPC-DS: Total hot run time: 187015 ms |
ClickBench: Total hot run time: 29.84 s |
|
run buildall |
TPC-H: Total hot run time: 33901 ms |
TPC-DS: Total hot run time: 193904 ms |
ClickBench: Total hot run time: 29.78 s |
|
run buildall |
|
run buildall |
FE UT Coverage ReportIncrement line coverage |
TPC-H: Total hot run time: 31965 ms |
TPC-DS: Total hot run time: 173427 ms |
ClickBench: Total hot run time: 26.51 s |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 34 out of 36 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
run buildall |
TPC-DS: Total hot run time: 0 ms |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
FE Regression Coverage ReportIncrement line coverage |
f03e812 to
3572829
Compare
|
run buildall |
TPC-DS: Total hot run time: 0 ms |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
FE Regression Coverage ReportIncrement line coverage |
What problem does this PR solve?
Issue Number: close #xxx
Related PR: bug introduced by #44102, expanded by #45256, partially fixed by #47911
Problem Summary:
also fixed Auto Partition + Auto Increment-Column may core dump in loading
Release note
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)