Github user tillrohrmann commented on a diff in the pull request:
https://github.com/apache/flink/pull/6284#discussion_r202082557
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/clusterframework/BootstrapTools.java
---
@@ -467,4 +471,22 @@ public static String getStartCommand(String template,
}
return template;
}
+
+ /**
+ * Set temporary configuration directories if necesary
+ *
+ * @param configuration flink config to patch
+ * @param defaultDirs in case no tmp directories is set, next
directories will be applied
+ */
+ public static void setTmpDirectoriesConfig(Configuration configuration,
String defaultDirs){
+ if (configuration.contains(CoreOptions.TMP_DIRS) &&
!configuration.getString(CoreOptions.TMP_DIRS).isEmpty()) {
+ LOG.info("Overriding Fink's temporary file directories
with those " +
+ "specified in the Flink config: " +
configuration.getValue(CoreOptions.TMP_DIRS));
--- End diff --
Should use logging placeholder `{}`
---
