wangyang0918 commented on a change in pull request #9984:
[FLINK-9495][kubernetes] Implement ResourceManager for Kubernetes.
URL: https://github.com/apache/flink/pull/9984#discussion_r354675578
##########
File path:
flink-runtime/src/main/java/org/apache/flink/runtime/clusterframework/BootstrapTools.java
##########
@@ -610,4 +613,32 @@ public Config getAkkaConfig() {
return AkkaUtils.getThreadPoolExecutorConfig(this);
}
}
+
+ /**
+ * Get dynamic properties based on two Flink configuration. If base
config does not contain and target config
+ * contains the key or the value is different, it should be added to
results. Otherwise, if the base config contains
+ * and target config does not contain the key, it will be ignored.
+ * @param baseConfig The base configuration.
+ * @param targetConfig The target configuration.
+ * @return Dynamic properties as string, separated by space.
+ */
+ public static String getDynamicProperties(
+ org.apache.flink.configuration.Configuration baseConfig,
Review comment:
Make sense. I will fix it.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
