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

Posted to issues@flink.apache.org by GJL <gi...@git.apache.org> on 2018/02/27 10:49:09 UTC

[GitHub] flink pull request #5584: [FLINK-8787][flip6] WIP Deploying FLIP-6 YARN sess...

Github user GJL commented on a diff in the pull request:

    https://github.com/apache/flink/pull/5584#discussion_r170882807
  
    --- Diff: flink-yarn/src/main/java/org/apache/flink/yarn/AbstractYarnClusterDescriptor.java ---
    @@ -186,7 +187,7 @@ public YarnClient getYarnClient() {
     	protected abstract String getYarnJobClusterEntrypoint();
     
     	public Configuration getFlinkConfiguration() {
    -		return flinkConfiguration;
    +		return new UnmodifiableConfiguration(flinkConfiguration);
    --- End diff --
    
    fixed


---