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

    https://github.com/apache/flink/pull/6284#discussion_r202084291
  
    --- 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){
    --- End diff --
    
    Should we maybe call it `updateTmpDirectoriesInConfiguration`?


---

Reply via email to