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

    https://github.com/apache/flink/pull/6284#discussion_r202321585
  
    --- 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 --
    
    done


---

Reply via email to