You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@dolphinscheduler.apache.org by GitBox <gi...@apache.org> on 2022/08/18 12:39:05 UTC

[GitHub] [dolphinscheduler] lenboo commented on a diff in pull request #11541: Add dependent task instance log

lenboo commented on code in PR #11541:
URL: https://github.com/apache/dolphinscheduler/pull/11541#discussion_r949081603


##########
dolphinscheduler-master/src/main/java/org/apache/dolphinscheduler/server/master/runner/task/DependentTaskProcessor.java:
##########
@@ -173,6 +174,10 @@ private boolean allDependentTaskFinish() {
                 finish = false;
             }
         }
+        if (!finish) {
+            // todo: add information, which dependent doesn't finished

Review Comment:
   Maybe there are many unused log when the dependent task is running.
   I think we only need to add the log :
   1.  all the dependent data 
   2.  the finished dependent tasks 



-- 
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: commits-unsubscribe@dolphinscheduler.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org