This is an automated email from the ASF dual-hosted git repository. dataroaring pushed a commit to branch branch-3.0 in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/branch-3.0 by this push: new a88ef51f38d [fix](pipeline) Do not schedule finalized task again (#49007) a88ef51f38d is described below commit a88ef51f38db560d4958a2632cad4de08a0ce635 Author: Gabriel <liwenqi...@selectdb.com> AuthorDate: Mon Mar 17 10:13:24 2025 +0800 [fix](pipeline) Do not schedule finalized task again (#49007) --- be/src/pipeline/task_scheduler.cpp | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/be/src/pipeline/task_scheduler.cpp b/be/src/pipeline/task_scheduler.cpp index 475d3a8065f..1fb3fbb3c36 100644 --- a/be/src/pipeline/task_scheduler.cpp +++ b/be/src/pipeline/task_scheduler.cpp @@ -76,10 +76,6 @@ void _close_task(PipelineTask* task, Status exec_status) { // Should count the memory to the query or the query's memory will not decrease when part of // task finished. SCOPED_ATTACH_TASK(task->runtime_state()); - if (task->is_finalized()) { - task->set_running(false); - return; - } // close_a_pipeline may delete fragment context and will core in some defer // code, because the defer code will access fragment context it self. auto lock_for_context = task->fragment_context()->shared_from_this(); @@ -107,6 +103,9 @@ void TaskScheduler::_do_work(size_t index) { static_cast<void>(_task_queue->push_back(task, index)); continue; } + if (task->is_finalized()) { + continue; + } task->log_detail_if_need(); task->set_running(true); task->set_task_queue(_task_queue.get()); --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org