-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
[fix](spill) should call set_ready after changing the status #40075
base: master
Are you sure you want to change the base?
Conversation
Thank you for your contribution to Apache Doris. Since 2024-03-18, the Document has been moved to doris-website. |
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
TPC-H: Total hot run time: 38159 ms
|
TPC-DS: Total hot run time: 192550 ms
|
ClickBench: Total hot run time: 32.29 s
|
run buildall |
TPC-H: Total hot run time: 38093 ms
|
TPC-DS: Total hot run time: 188343 ms
|
ClickBench: Total hot run time: 32.85 s
|
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
TPC-H: Total hot run time: 38474 ms
|
TPC-DS: Total hot run time: 188831 ms
|
ClickBench: Total hot run time: 32.42 s
|
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
TPC-H: Total hot run time: 38203 ms
|
TPC-DS: Total hot run time: 195710 ms
|
ClickBench: Total hot run time: 32.53 s
|
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
TPC-H: Total hot run time: 37368 ms
|
TPC-DS: Total hot run time: 193346 ms
|
ClickBench: Total hot run time: 31.73 s
|
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
TPC-H: Total hot run time: 38245 ms
|
TPC-DS: Total hot run time: 193247 ms
|
ClickBench: Total hot run time: 32.08 s
|
PR approved by at least one committer and no changes requested. |
PR approved by anyone and no changes requested. |
Proposed changes
After
set_ready
is called, other threads will access theStatus
variable, so to avoid conflicts, we should callset_ready
after settingStatus
.