Rubik-W opened a new issue #2722:
URL: https://github.com/apache/incubator-dolphinscheduler/issues/2722


   Hello!
   
   Abort dependent task check rules, I think the current implementation is 
problematic.
   
   E.g. check logic: this week
   The current implementation:
   1.get this Monday
   2.the date list of this Monday to current date(asc order)
   3.Iterate through the list of dates in sequence to check the status of all 
process instances. There is a failure, the ultimate is failure state.
   
   I think, the check logic should be:
   1.get date range, start date and end date
   2.if type is DEPENDENT_ALL
        query the first process instance in reverse order, check this instance 
state.
   3.if type is non DEPENDENT_ALL
        query the first task instance in reverse order, check this instance 
state.


----------------------------------------------------------------
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]


Reply via email to