flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [flink] walterddr commented on a change in pull request #10193: [FLINK-13938][yarn] Use pre-uploaded flink binary to accelerate flink submission
Date Wed, 27 Nov 2019 00:34:06 GMT
walterddr commented on a change in pull request #10193: [FLINK-13938][yarn] Use pre-uploaded
flink binary to accelerate flink submission
URL: https://github.com/apache/flink/pull/10193#discussion_r351045001
 
 

 ##########
 File path: flink-yarn/src/main/java/org/apache/flink/yarn/Utils.java
 ##########
 @@ -263,6 +298,36 @@ private static LocalResource registerLocalResource(FileSystem fs, Path
remoteRsr
 		return localResource;
 	}
 
+	/**
+	 * Register a local resource with resource info. The resource info may contains multiple
parts.
+	 * For example, RemotePath;[resourceSize;resourceModificationTime;LocalResourceVisibility]
+	 * @param resourceInfoStr resource info string
+	 * @param yarnConfig yarn configuration
+	 * @return local resource tuple, f0 is filename, f1 is local resource.
+	 */
+	private static Tuple2<String, LocalResource> registerLocalResource(
 
 Review comment:
   @TisonKun: sorry could you elaborate the "in-place" pattern you mentioned? and are you
a +0 on the 1) current PR implementation, or 2) on the split to `registerLocal...` & `registerPreloaded...`
?

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

Mime
View raw message