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) <[email protected]>
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: [email protected]
For additional commands, e-mail: [email protected]
