Github user suez1224 commented on a diff in the pull request:
https://github.com/apache/flink/pull/5416#discussion_r175341644
--- 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 --
In secure mode, the test will append keytab and principal option to the
on-disk flink-conf.yaml before starting. So using a vanilla Configuration
object is actually one of the issues why the current test code is broken.
---
