zentol commented on a change in pull request #19047:
URL: https://github.com/apache/flink/pull/19047#discussion_r824693430



##########
File path: 
flink-runtime/src/main/java/org/apache/flink/runtime/dispatcher/Dispatcher.java
##########
@@ -428,6 +428,12 @@ private void stopDispatcherServices() throws Exception {
 
         try {
             if (isDuplicateJob(jobGraph.getJobID())) {
+                if (isGloballyTerminatedAndCleanedUp(jobGraph.getJobID())) {
+                    log.warn(
+                            "A job having the ID {} is already in 
globally-terminal state and marked as clean in the JobResultStore. No execution 
is going to happen.",
+                            jobGraph.getJobID());

Review comment:
       Essentially, I'm not sure if users know what a globally-terminal state 
is, nor am I sure whether the act of cleaning is something they're interested 
in.




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