[GitHub] [dolphinscheduler] yutaoChina commented on a change in pull request #5561: yarn application state check add a Retrial mechanism

Tue, 07 Sep 2021 00:02:37 -0700 


yutaoChina commented on a change in pull request #5561:
URL: https://github.com/apache/dolphinscheduler/pull/5561#discussion_r703234941



##########
File path: 
dolphinscheduler-server/src/main/java/org/apache/dolphinscheduler/server/worker/task/AbstractCommandExecutor.java
##########
@@ -428,9 +431,27 @@ public boolean isSuccessOfYarnState(List<String> appIds) {
             result = false;
         }
         return result;
-
     }
-
+    
+    /**
+     *
+     * @param appId
+     * @return appId ExecutionStatus
+     * @throws Exception
+     */
+    private ExecutionStatus retryExecutionStatus(String appId,ExecutionStatus 
applicationStatus) throws Exception {
+        for (int i = 0; i < 10; i++) {
+            applicationStatus = 
HadoopUtils.getInstance().getApplicationStatus(appId);
+            if (applicationStatus.equals(ExecutionStatus.FAILURE)
+                    || applicationStatus.equals(ExecutionStatus.KILL)) {

Review comment:
       
![image](https://user-images.githubusercontent.com/9714395/132296305-661d6c6b-c722-4089-87a3-d690f38eb246.png)
   just like this!
   then retry 10 time Still failureļ¼Œso it failed




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to