Uh oh!
There was an error while loading. Please reload this page.
[opt](MultiCast) Avoid copying while holding a lock - #37462
Conversation
doris-robot
commented
Jul 8, 2024
Thank you for your contribution to Apache Doris. Since 2024-03-18, the Document has been moved to doris-website. |
Mryange
commented
Jul 8, 2024
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
doris-robot
commented
Jul 8, 2024
TPC-H: Total hot run time: 39491 ms |
Mryange
commented
Jul 8, 2024
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
doris-robot
commented
Jul 8, 2024
TPC-H: Total hot run time: 40198 ms |
Mryange
commented
Jul 8, 2024
run performance |
doris-robot
commented
Jul 8, 2024
TPC-H: Total hot run time: 39591 ms |
Mryange
commented
Jul 8, 2024
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
doris-robot
commented
Jul 8, 2024
TPC-H: Total hot run time: 40561 ms |
Mryange
commented
Jul 8, 2024
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
doris-robot
commented
Jul 8, 2024
TPC-H: Total hot run time: 39372 ms |
doris-robot
commented
Jul 8, 2024
TPC-DS: Total hot run time: 173064 ms |
doris-robot
commented
Jul 8, 2024
ClickBench: Total hot run time: 30.37 s |
Mryange
commented
Jul 9, 2024
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
doris-robot
commented
Jul 9, 2024
TPC-H: Total hot run time: 40251 ms |
doris-robot
commented
Jul 9, 2024
TPC-DS: Total hot run time: 172148 ms |
doris-robot
commented
Jul 9, 2024
ClickBench: Total hot run time: 30.55 s |
Mryange
commented
Jul 9, 2024
run p0 |
Mryange
commented
Jul 9, 2024
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
Mryange
commented
Jul 10, 2024
run buildall |
doris-robot
commented
Jul 10, 2024
TPC-H: Total hot run time: 40394 ms |
doris-robot
commented
Jul 10, 2024
TPC-DS: Total hot run time: 173064 ms |
doris-robot
commented
Jul 10, 2024
ClickBench: Total hot run time: 31.14 s |
| void MultiCastDataStreamer::_wait_copy_block(vectorized::Block* block, int& un_finish_copy) { | ||
| std::unique_lock l(_mutex); | ||
| _cv.wait(l, [&]() { return un_finish_copy == 0; }); |
There was a problem hiding this comment.
danger will notify different queue last
PR approved by at least one committer and no changes requested. |
PR approved by anyone and no changes requested. |
Previously, copying was done while holding a lock; Now, get block while holding the lock and then copy
Previously, copying was done while holding a lock; Now, get block while holding the lock and then copy
pick #37462 The difference is quite large, so it can't be directly picked.
pick apache#37462 The difference is quite large, so it can't be directly picked.
Previously, copying was done while holding a lock; Now, get block while holding the lock and then copy
Previously, copying was done while holding a lock; Now, get block while holding the lock and then copy
Proposed changes
Previously, copying was done while holding a lock;
Now, get block while holding the lock and then copy