Uh oh!
There was an error while loading. Please reload this page.
[fix](group commit) reduce cpu cost for group_commit get_block - #49822
Conversation
Thearas
commented
Apr 7, 2025
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
a2f90ac to
3da47f3Comparemymeiyi
commented
Apr 7, 2025
run buildall |
doris-robot
commented
Apr 7, 2025
TPC-H: Total hot run time: 34367 ms |
doris-robot
commented
Apr 7, 2025
TPC-DS: Total hot run time: 192238 ms |
doris-robot
commented
Apr 7, 2025
ClickBench: Total hot run time: 31.65 s |
mymeiyi
commented
Apr 7, 2025
run p0 |
hello-stephen
commented
Apr 7, 2025
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
3da47f3 to
5103a6eComparemymeiyi
commented
Apr 7, 2025
run buildall |
doris-robot
commented
Apr 7, 2025
TPC-H: Total hot run time: 34214 ms |
doris-robot
commented
Apr 7, 2025
TPC-DS: Total hot run time: 193538 ms |
doris-robot
commented
Apr 7, 2025
ClickBench: Total hot run time: 30.91 s |
Uh oh!
There was an error while loading. Please reload this page.
doris-robot
commented
Apr 7, 2025
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
ba2cbcb to
a023e77Comparemymeiyi
commented
Apr 11, 2025
run buildall |
doris-robot
commented
Apr 11, 2025
TPC-H: Total hot run time: 35526 ms |
doris-robot
commented
Apr 11, 2025
TPC-DS: Total hot run time: 193284 ms |
doris-robot
commented
Apr 11, 2025
ClickBench: Total hot run time: 31.18 s |
hello-stephen
commented
Apr 11, 2025
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
mymeiyi
commented
Apr 11, 2025
run cloud_p0 |
1 similar comment
mymeiyi
commented
Apr 11, 2025
run cloud_p0 |
| _runtime_filter_timer = std::make_shared<pipeline::RuntimeFilterTimer>( | ||
| MonotonicMillis(), load_block_queue->get_group_commit_interval_ms(), | ||
| _timer_dependency); | ||
| std::vector<std::shared_ptr<pipeline::RuntimeFilterTimer>> timers; |
There was a problem hiding this comment.
group commit需要按照提交时间被唤醒,现在只有runtime filter支持这种
There was a problem hiding this comment.
这种地方都得用注释写清楚,否则,别人还以为我们的group commit 跟runtime filter 有啥关联
a023e77 to
18509efComparemymeiyi
commented
May 19, 2025
run buildall |
doris-robot
commented
May 19, 2025
TPC-H: Total hot run time: 33561 ms |
doris-robot
commented
May 19, 2025
TPC-DS: Total hot run time: 192651 ms |
doris-robot
commented
May 19, 2025
ClickBench: Total hot run time: 28.74 s |
hello-stephen
commented
May 19, 2025
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
hello-stephen
commented
May 19, 2025
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
hello-stephen
commented
May 19, 2025
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
PR approved by at least one committer and no changes requested. |
PR approved by anyone and no changes requested. |
…mit get_block apache#49822 (apache#51079)" This reverts commit 212cd8f.
…roup_commit get_block apache#49822 (apache#51079)"" This reverts commit e79a574.
### What problem does this PR solve? 1. after #49822, the group_commit_scan_operator is blocked by time dependency until timeout, this pr fix it 2. modify some debug log 3. print the create plan error to client
1. after apache#49822, the group_commit_scan_operator is blocked by time dependency until timeout, this pr fix it 2. modify some debug log 3. print the create plan error to client
### What problem does this PR solve? 1. after apache#49822, the group_commit_scan_operator is blocked by time dependency until timeout, this pr fix it 2. modify some debug log 3. print the create plan error to client
What problem does this PR solve?
Issue Number: close #xxx
Related PR: #xxx
Problem Summary:
Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)