You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@flink.apache.org by GitBox <gi...@apache.org> on 2022/01/12 12:51:28 UTC

[GitHub] [flink] dawidwys commented on a change in pull request #18332: [FLINK-25427] Fix SavepointITCase.testTriggerSavepointAndResumeWithNo…

dawidwys commented on a change in pull request #18332:
URL: https://github.com/apache/flink/pull/18332#discussion_r783045457



##########
File path: flink-runtime/src/main/java/org/apache/flink/runtime/minicluster/MiniCluster.java
##########
@@ -1175,6 +1185,31 @@ private void terminateMiniClusterServices() throws Exception {
         }
     }
 
+    /**
+     * Prevent multiple submission of the same JobGraph that has been mutated in between

Review comment:
       Is that really a contract we want to enforce? Why are you concerned with mutating the `JobGraph` between submissions? If I understand it correctly, it would be safe to do so with a proper cluster.




-- 
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: issues-unsubscribe@flink.apache.org

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