ruanwenjun commented on code in PR #12219:
URL: https://github.com/apache/dolphinscheduler/pull/12219#discussion_r991420500
##########
dolphinscheduler-common/src/main/java/org/apache/dolphinscheduler/common/utils/DateUtils.java:
##########
@@ -575,6 +575,12 @@ public static long getRemainTime(Date baseTime, long
intervalSeconds) {
return 0;
}
long usedTime = (System.currentTimeMillis() - baseTime.getTime()) /
1000;
+ // ignore worker's clock later than master nodes.
Review Comment:
It's better don't add this in this method, this is Utils. And I don't think
we need to deal with this, we only need to announce the clock should sync in
master/worker, since this change don't solve the problem when the clock is not
sync.
--
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]
