Github user dawidwys commented on a diff in the pull request:
https://github.com/apache/flink/pull/5846#discussion_r191133512
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/dispatcher/Dispatcher.java
---
@@ -255,6 +264,35 @@ public void start() throws Exception {
}
}
+ /**
+ *
+ * @param job job graph
+ * @param configuration configuration in flink-conf.yaml
+ * @param isClusterPoint whether this is client side or cluster site
+ */
+ public static void setJobgraphRestartStrategy(JobGraph job,
Configuration configuration, boolean isClusterPoint) {
--- End diff --
Don't think it is the right place. Especially as it is used also on the
client side. Maybe move to `JobGraph`?
---
