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

wenjun 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 590d16f22c [Fix-17473] [Task Plugin] Fix Serverless Spark Task final 
state incorrect for failure (#17475)
590d16f22c is described below

commit 590d16f22c0238142328f9ef3a2a90aa3cbac0a7
Author: Eric Gao <[email protected]>
AuthorDate: Thu Sep 4 23:26:25 2025 +0800

    [Fix-17473] [Task Plugin] Fix Serverless Spark Task final state incorrect 
for failure (#17475)
---
 .../task/aliyunserverlessspark/AliyunServerlessSparkTask.java  |  2 +-
 .../aliyunserverlessspark/AliyunServerlessSparkTaskTest.java   | 10 ++++++++++
 2 files changed, 11 insertions(+), 1 deletion(-)

diff --git 
a/dolphinscheduler-task-plugin/dolphinscheduler-task-aliyunserverlessspark/src/main/java/org/apache/dolphinscheduler/plugin/task/aliyunserverlessspark/AliyunServerlessSparkTask.java
 
b/dolphinscheduler-task-plugin/dolphinscheduler-task-aliyunserverlessspark/src/main/java/org/apache/dolphinscheduler/plugin/task/aliyunserverlessspark/AliyunServerlessSparkTask.java
index 8824665180..c66cc23790 100644
--- 
a/dolphinscheduler-task-plugin/dolphinscheduler-task-aliyunserverlessspark/src/main/java/org/apache/dolphinscheduler/plugin/task/aliyunserverlessspark/AliyunServerlessSparkTask.java
+++ 
b/dolphinscheduler-task-plugin/dolphinscheduler-task-aliyunserverlessspark/src/main/java/org/apache/dolphinscheduler/plugin/task/aliyunserverlessspark/AliyunServerlessSparkTask.java
@@ -197,7 +197,7 @@ public class AliyunServerlessSparkTask extends 
AbstractRemoteTask {
             case Success:
                 return TaskConstants.EXIT_CODE_SUCCESS;
             case Failed:
-                return TaskConstants.EXIT_CODE_KILL;
+                return TaskConstants.EXIT_CODE_FAILURE;
             default:
                 return TaskConstants.EXIT_CODE_FAILURE;
         }
diff --git 
a/dolphinscheduler-task-plugin/dolphinscheduler-task-aliyunserverlessspark/src/test/java/org/apache/dolphinscheduler/plugin/task/aliyunserverlessspark/AliyunServerlessSparkTaskTest.java
 
b/dolphinscheduler-task-plugin/dolphinscheduler-task-aliyunserverlessspark/src/test/java/org/apache/dolphinscheduler/plugin/task/aliyunserverlessspark/AliyunServerlessSparkTaskTest.java
index 6a29356fa7..74547497af 100644
--- 
a/dolphinscheduler-task-plugin/dolphinscheduler-task-aliyunserverlessspark/src/test/java/org/apache/dolphinscheduler/plugin/task/aliyunserverlessspark/AliyunServerlessSparkTaskTest.java
+++ 
b/dolphinscheduler-task-plugin/dolphinscheduler-task-aliyunserverlessspark/src/test/java/org/apache/dolphinscheduler/plugin/task/aliyunserverlessspark/AliyunServerlessSparkTaskTest.java
@@ -222,4 +222,14 @@ public class AliyunServerlessSparkTaskTest {
         verify(mockAliyunServerlessSparkParameters).getEngineReleaseVersion();
         verify(mockAliyunServerlessSparkParameters).isProduction();
     }
+
+    @Test
+    public void testMapFinalStateToExitCode() {
+        Assertions.assertEquals(TaskConstants.EXIT_CODE_SUCCESS,
+                
aliyunServerlessSparkTask.mapFinalStateToExitCode(RunState.Success));
+        Assertions.assertEquals(TaskConstants.EXIT_CODE_FAILURE,
+                
aliyunServerlessSparkTask.mapFinalStateToExitCode(RunState.Failed));
+        Assertions.assertEquals(TaskConstants.EXIT_CODE_FAILURE,
+                
aliyunServerlessSparkTask.mapFinalStateToExitCode(RunState.Cancelled));
+    }
 }

Reply via email to