yanghua commented on a change in pull request #10187: [FLINK-13938][yarn]
Enable configure shared libraries on YARN
URL: https://github.com/apache/flink/pull/10187#discussion_r346647670
##########
File path:
flink-yarn/src/main/java/org/apache/flink/yarn/cli/FlinkYarnSessionCli.java
##########
@@ -194,6 +195,7 @@ public FlinkYarnSessionCli(
applicationId = new Option(shortPrefix + "id", longPrefix +
"applicationId", true, "Attach to running YARN session");
queue = new Option(shortPrefix + "qu", longPrefix + "queue",
true, "Specify YARN queue.");
shipPath = new Option(shortPrefix + "t", longPrefix + "ship",
true, "Ship files in the specified directory (t for transfer)");
+ sharedLibs = new Option(shortPrefix + "sl", longPrefix +
"sharedLibs", true, "Upload a copy of flink libs beforehand and specify the
path to use public visibility feature of YARN NodeManager localizing
resources.");
Review comment:
Can we add a test case about this config option or the whole feature like
ship files?
----------------------------------------------------------------
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
