branch-4.0: [fix](be) Fix time-sharing executor queued task count#65043
branch-4.0: [fix](be) Fix time-sharing executor queued task count#65043Jungzhang wants to merge 2 commits into
Conversation
Keep the executor queued-task metric consistent when queued splits are removed before execution. The fix routes queue offer/remove operations through helpers that update _total_queued_tasks together with the split queue and token state. Add a regression test that enqueues splits with no worker threads, removes their tasks, and verifies the queue count returns to zero so later submissions are not rejected as full.
|
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
|
run buildall |
|
run p0 |
|
run external |
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
|
run p0 |
|
run external |
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
|
Hi @morningman PR #65043 is ready for review. The relevant formatting and compilation checks have passed. Could you please review and merge it when convenient? Thank you! |
|
run buildall |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
What problem does this PR solve?
Issue Number: close #xxx
Related PR: #63568
Problem Summary:
Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)