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_r346647799
##########
File path:
flink-yarn/src/main/java/org/apache/flink/yarn/configuration/YarnConfigOptions.java
##########
@@ -215,6 +215,14 @@
.noDefaultValue()
.withDescription("A semicolon-separated list of
directories to be shipped to the YARN cluster.");
+ public static final ConfigOption<List<String>> SHARED_LIBRARIES =
+ key("yarn.shared-libraries")
+ .stringType()
+ .asList()
+ .noDefaultValue()
+ .withDescription("specify the path to use
public visibility feature of " +
Review comment:
Also, shall we capitalize the first letter? Like other config option
description.
----------------------------------------------------------------
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
