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

    https://github.com/apache/flink/pull/5416#discussion_r175292394
  
    --- Diff: 
flink-yarn-tests/src/test/java/org/apache/flink/yarn/YarnTestBase.java ---
    @@ -206,7 +209,7 @@ public void checkClusterEmpty() throws IOException, 
YarnException {
                        }
                }
     
    -           flinkConfiguration = new 
org.apache.flink.configuration.Configuration();
    +           flinkConfiguration = GlobalConfiguration.loadConfiguration();
    --- End diff --
    
    Why do we have to load the configuration from disk and cannot use a vanilla 
`Configuration` object?


---

Reply via email to