This is an automated email from the ASF dual-hosted git repository.

dataroaring pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/master by this push:
     new 2f795bc21dc [fix](export) remove task from map finally (#51171)
2f795bc21dc is described below

commit 2f795bc21dc9c84b2fc20b6fbe3ba13687769552
Author: Mingyu Chen (Rayner) <morning...@163.com>
AuthorDate: Mon May 26 10:26:26 2025 +0800

    [fix](export) remove task from map finally (#51171)
    
    ### What problem does this PR solve?
    
    Related PR: #47974
    
    Problem Summary:
    
    When cancel task, the task should be removed from `taskExecutorMap`,
    otherwise, if `tryPublishTask()` failed when `addMemoryTask`, the task
    will
    be leaked in `taskExecutorMap`
---
 .../java/org/apache/doris/scheduler/manager/TransientTaskManager.java   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/fe/fe-core/src/main/java/org/apache/doris/scheduler/manager/TransientTaskManager.java
 
b/fe/fe-core/src/main/java/org/apache/doris/scheduler/manager/TransientTaskManager.java
index 5c62caede9c..feaf3365a35 100644
--- 
a/fe/fe-core/src/main/java/org/apache/doris/scheduler/manager/TransientTaskManager.java
+++ 
b/fe/fe-core/src/main/java/org/apache/doris/scheduler/manager/TransientTaskManager.java
@@ -63,7 +63,7 @@ public class TransientTaskManager {
     }
 
     public void cancelMemoryTask(Long taskId) throws JobException {
-        TransientTaskExecutor transientTaskExecutor = 
taskExecutorMap.get(taskId);
+        TransientTaskExecutor transientTaskExecutor = 
taskExecutorMap.remove(taskId);
         if (transientTaskExecutor != null) {
             transientTaskExecutor.cancel();
         }


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to