jerrypeng commented on a change in pull request #1956: Add function-package-url support in function cli add url support to function cli URL: https://github.com/apache/incubator-pulsar/pull/1956#discussion_r194959000
########## File path: pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CmdFunctions.java ########## @@ -209,11 +217,12 @@ void processArguments() throws Exception { protected String functionName; @Parameter(names = "--className", description = "The function's class name") protected String className; - @Parameter( - names = "--jar", - description = "Path to the jar file for the function (if the function is written in Java)", - listConverter = StringConverter.class) + @Parameter(names = "--jar", description = "Path to the jar file for the function (if the function is written in Java). It also supports url-path (http/https/file) from which worker can download the package.", listConverter = StringConverter.class) protected String jarFile; + + @Parameter(names = "--classNameArgType", description = "Function impl class's argument type if jar path is url (http/file).") Review comment: please see comment in CmdSource ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services