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

    https://github.com/apache/flink/pull/4939#discussion_r149070583
  
    --- Diff: flink-yarn/src/main/java/org/apache/flink/yarn/Utils.java ---
    @@ -117,27 +118,50 @@ public static void setupYarnClassPath(Configuration 
conf, Map<String, String> ap
        }
     
        /**
    +    * Copy a local file to a remote file system.
    +    *
    +    * @param fs
    +    *              remote filesystem
    +    * @param appId
    +    *              application ID
    +    * @param localRsrcPath
    +    *              path to the local file
    +    * @param homedir
    +    *              remote home directory base (will be extended)
    +    * @param relativeTargetPath
    +    *              relative target path of the file (will be prefixed be 
the full home directory we set up)
    +    *
         * @return Path to remote file (usually hdfs)
    -    * @throws IOException
         */
    -   public static Path setupLocalResource(
    -                   FileSystem fs,
    -                   String appId, Path localRsrcPath,
    -                   LocalResource appMasterJar,
    -                   Path homedir) throws IOException {
    +   static Tuple2<Path, LocalResource> setupLocalResource(
    +           FileSystem fs,
    +           String appId,
    +           Path localRsrcPath,
    --- End diff --
    
    `localSrcPath`?


---

Reply via email to