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)


---

Reply via email to