cuspymd commented on a change in pull request #4099:
URL: https://github.com/apache/zeppelin/pull/4099#discussion_r617569641



##########
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:
       There are several properties with similar names, so it seems difficult 
to understand the difference between them without checking how they are used in 
the actual code from the user's point of view.
   - flink.udf.jars
   - flink.udf.jars.packages
   - flink.udf.packages
   - flink.udf.packages.packages




-- 
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:
us...@infra.apache.org


Reply via email to