[GitHub] [zeppelin] Jiabao-Sun commented on a change in pull request #4099: [ZEPPELIN-5332] Add configuration of flink.udf.packages and flink.udf.packages.packages

[GitBox <gi...@apache.org>]

Jiabao-Sun commented on a change in pull request #4099:
URL: https://github.com/apache/zeppelin/pull/4099#discussion_r617670254



##########
File path: docs/interpreter/flink.md
##########
@@ -155,6 +155,16 @@ You can also add and set other flink properties which are not listed in the tabl
     <td></td>
     <td>Packages (comma separated) that would be searched for the udf defined in `flink.udf.jars`.</td>
   </tr>
+  <tr>
+    <td>flink.udf.packages</td>
+    <td></td>
+    <td>Flink udf packages (comma separated), zeppelin will register udf in this jar automatically for user. These udf packages could be found in the maven central or private repository. The udf name is the class name. e.g. org.apache.flink:flink-udfs-demo0:1.10.0,org.apache.flink:flink-udfs-demo1:1.10.0</td>
+  </tr>
+  <tr>
+    <td>flink.udf.packages.packages</td>

Review comment:
       This is true, but the basis for naming is to refer to fink.udf.packages.
   It may cause misunderstandings and compatibility issues with the old code, if we change this naming style.
   
   If there is a more suitable name, I hope to get your suggestions.




-- 
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