Github user GJL commented on a diff in the pull request:
https://github.com/apache/flink/pull/5584#discussion_r170679952
--- 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 --
not needed, `flinkConfiguration` is unmodifiable already
---
