Github user dawidwys commented on a diff in the pull request:
https://github.com/apache/flink/pull/5846#discussion_r191140799
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/dispatcher/Dispatcher.java
---
@@ -250,6 +259,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) {
+
+ try {
+ ExecutionConfig config =
job.getSerializedExecutionConfig().deserializeValue(
+ Dispatcher.class.getClassLoader());
--- End diff --
You should use userclassloader here (especially in case of cluster side)
---