Uh oh!
There was an error while loading. Please reload this page.
[fix](blocking queue) introduce condition variable wait timeout to avoid blocking queue deadlock - #50906
Conversation
Thearas
commented
May 14, 2025
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
sollhui
commented
May 14, 2025
run buildall |
doris-robot
commented
May 14, 2025
TPC-H: Total hot run time: 34162 ms |
doris-robot
commented
May 14, 2025
TPC-DS: Total hot run time: 186542 ms |
doris-robot
commented
May 14, 2025
ClickBench: Total hot run time: 29.11 s |
doris-robot
commented
May 14, 2025
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
hello-stephen
commented
May 14, 2025
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
sollhui
commented
May 15, 2025
run buildall |
sollhui
commented
May 15, 2025
run buildall |
PR approved by at least one committer and no changes requested. |
PR approved by anyone and no changes requested. |
c669056 to
8662e67Comparedataroaring
commented
May 18, 2025
run buildall |
Uh oh!
There was an error while loading. Please reload this page.
sollhui
commented
May 26, 2025
run buildall |
hello-stephen
commented
May 26, 2025
Cloud UT Coverage ReportIncrement line coverage Increment coverage report
|
sollhui
commented
May 26, 2025
run buildall |
1 similar comment
sollhui
commented
May 26, 2025
run buildall |
sollhui
commented
Jun 5, 2025
run buildall |
sollhui
commented
Jun 5, 2025
run buildall |
sollhui
commented
Jun 5, 2025
run buildall |
doris-robot
commented
Jun 5, 2025
TPC-H: Total hot run time: 33988 ms |
doris-robot
commented
Jun 5, 2025
TPC-DS: Total hot run time: 192627 ms |
doris-robot
commented
Jun 5, 2025
ClickBench: Total hot run time: 28.64 s |
sollhui
commented
Jun 6, 2025
run buildall |
sollhui
commented
Jun 9, 2025
run buildall |
doris-robot
commented
Jun 9, 2025
TPC-H: Total hot run time: 33907 ms |
doris-robot
commented
Jun 9, 2025
TPC-DS: Total hot run time: 186987 ms |
doris-robot
commented
Jun 9, 2025
ClickBench: Total hot run time: 28.88 s |
doris-robot
commented
Jun 9, 2025
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
hello-stephen
commented
Jun 9, 2025
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
PR approved by at least one committer and no changes requested. |
…oid blocking queue deadlock (#50906) ### What problem does this PR solve? We found some routine load tasks are never finish, and these task hang for **blocking queue** is deadlock. We observed the stack using GDB and printed the values:   It is clear that both the get thread and put thread are waiting on the condition variable at the same time, causing a deadlock. However, unfortunately, we have not located the problem yet. we choose introducing timeout to avoid blocking queue deadlock temporarily.If anyone encounters this problem or has any ideas, please feel free to discuss under this PR.
…oid blocking queue deadlock (#50906) ### What problem does this PR solve? We found some routine load tasks are never finish, and these task hang for **blocking queue** is deadlock. We observed the stack using GDB and printed the values:   It is clear that both the get thread and put thread are waiting on the condition variable at the same time, causing a deadlock. However, unfortunately, we have not located the problem yet. we choose introducing timeout to avoid blocking queue deadlock temporarily.If anyone encounters this problem or has any ideas, please feel free to discuss under this PR.
What problem does this PR solve?
We found some routine load tasks are never finish, and these task hang for blocking queue is deadlock. We observed the stack using GDB and printed the values:


It is clear that both the get thread and put thread are waiting on the condition variable at the same time, causing a deadlock.
However, unfortunately, we have not located the problem yet. we choose introducing timeout to avoid blocking queue deadlock temporarily.If anyone encounters this problem or has any ideas, please feel free to discuss under this PR.
Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)