HuangZhenQiu commented on a change in pull request #8303: [FLINK-12343] 
[flink-yarn] add file replication config for yarn configuration
URL: https://github.com/apache/flink/pull/8303#discussion_r327931833
 
 

 ##########
 File path: 
flink-yarn/src/main/java/org/apache/flink/yarn/AbstractYarnClusterDescriptor.java
 ##########
 @@ -745,6 +745,17 @@ public ApplicationReport startAppMaster(
                // ship list that enables reuse of resources for task manager 
containers
                StringBuilder envShipFileList = new StringBuilder();
 
+               int fileReplication = 
yarnConfiguration.getInt(DFSConfigKeys.DFS_REPLICATION_KEY, 
DFSConfigKeys.DFS_REPLICATION_DEFAULT);
 
 Review comment:
   Good point out. It required integration with share cache client. I think it 
is probably out of the scope of the problem that the PR want to resolve. But we 
definitely can bring a further discussion.
   
    
https://hadoop.apache.org/docs/r3.1.1/hadoop-yarn/hadoop-yarn-site/SharedCache.html

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to