Rubik-W commented on issue #2112: [BUG] many dependent tasks depend on a node, 
task status bug
URL: 
https://github.com/apache/incubator-dolphinscheduler/issues/2112#issuecomment-596436852
 
 
   Yes, The processing of current task results and the submission of tasks are 
serial. I think this is a low probability event, instant optimization into 
parallel processing, it cannot absolutely solve the problem. Because at any 
moment when a task is submitting, there may be new failed tasks.
   
   I found another problem:
   When obtaining the active task execution result, if first task is success, 
second task is failure, failure strategy is end model. In the current code 
logic, subsequent nodes of successful tasks are added to the list to be 
submitted. 
   In business, no tasks should be added. And the end of the workflow should 
not go to sleep.
   
   Method updateProcessInstanceState() should be executed before method 
submitStandByTask().
   
   ```
               if(canSubmitTaskToQueue()){
                   submitStandByTask();
               }
               try {
                   Thread.sleep(Constants.SLEEP_TIME_MILLIS);
               } catch (InterruptedException e) {
                   logger.error(e.getMessage(),e);
               }
               updateProcessInstanceState();
   ```

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to