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

xincheng pushed a commit to branch dev
in repository https://gitbox.apache.org/repos/asf/dolphinscheduler.git


The following commit(s) were added to refs/heads/dev by this push:
     new 09afa75a3d [Improvement][Worker] No need to kill remote app when appid 
is empty (#14534)
09afa75a3d is described below

commit 09afa75a3d803281fa7f39b5f054a8fcbff1ab04
Author: Rick Cheng <[email protected]>
AuthorDate: Thu Jul 13 15:40:58 2023 +0800

    [Improvement][Worker] No need to kill remote app when appid is empty 
(#14534)
---
 .../org/apache/dolphinscheduler/plugin/task/api/utils/ProcessUtils.java  | 1 +
 1 file changed, 1 insertion(+)

diff --git 
a/dolphinscheduler-task-plugin/dolphinscheduler-task-api/src/main/java/org/apache/dolphinscheduler/plugin/task/api/utils/ProcessUtils.java
 
b/dolphinscheduler-task-plugin/dolphinscheduler-task-api/src/main/java/org/apache/dolphinscheduler/plugin/task/api/utils/ProcessUtils.java
index d60c007c36..130159137c 100644
--- 
a/dolphinscheduler-task-plugin/dolphinscheduler-task-api/src/main/java/org/apache/dolphinscheduler/plugin/task/api/utils/ProcessUtils.java
+++ 
b/dolphinscheduler-task-plugin/dolphinscheduler-task-api/src/main/java/org/apache/dolphinscheduler/plugin/task/api/utils/ProcessUtils.java
@@ -172,6 +172,7 @@ public final class ProcessUtils {
                 }
                 if (CollectionUtils.isEmpty(appIds)) {
                     log.info("The appId is empty");
+                    return;
                 }
                 ApplicationManager applicationManager = 
applicationManagerMap.get(ResourceManagerType.YARN);
                 applicationManager.killApplication(new 
YarnApplicationManagerContext(executePath, tenantCode, appIds));

Reply via email to