This is an automated email from the ASF dual-hosted git repository.

zjffdu pushed a commit to branch branch-0.10
in repository https://gitbox.apache.org/repos/asf/zeppelin.git


The following commit(s) were added to refs/heads/branch-0.10 by this push:
     new 058f38b  [ZEPPELIN-5524] Some configuration of flink interpreter are 
incorrect
058f38b is described below

commit 058f38b7430fa5e73c7682f86dea1ea518023f67
Author: Jeff Zhang <zjf...@apache.org>
AuthorDate: Thu Sep 16 11:18:50 2021 +0800

    [ZEPPELIN-5524] Some configuration of flink interpreter are incorrect
    
    ### What is this PR for?
    Type of `taskmanager.memory.process.size` & 
`jobmanager.memory.process.size` should be text.
    
    Default value of `zeppelin.flink.uiWebUrl` should be empty string.
    
    ### What type of PR is it?
    [Bug Fix ]
    
    ### Todos
    * [ ] - Task
    
    ### What is the Jira issue?
    * https://issues.apache.org/jira/browse/ZEPPELIN-5524
    
    ### How should this be tested?
    * CI pass
    
    ### Screenshots (if appropriate)
    
    ### Questions:
    * Does the licenses files need update? No
    * Is there breaking changes for older versions? No
    * Does this needs documentation? No
    
    Author: Jeff Zhang <zjf...@apache.org>
    
    Closes #4226 from zjffdu/ZEPPELIN-5524 and squashes the following commits:
    
    38494749b6 [Jeff Zhang] [ZEPPELIN-5524] Some configuration of flink 
interpreter are incorrect
    
    (cherry picked from commit fd7765a404890037aad7578bf7f4726bdeeacf0e)
    Signed-off-by: Jeff Zhang <zjf...@apache.org>
---
 .../flink-scala-parent/src/main/resources/interpreter-setting.json  | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git 
a/flink/flink-scala-parent/src/main/resources/interpreter-setting.json 
b/flink/flink-scala-parent/src/main/resources/interpreter-setting.json
index 12f41c4..dd67130 100644
--- a/flink/flink-scala-parent/src/main/resources/interpreter-setting.json
+++ b/flink/flink-scala-parent/src/main/resources/interpreter-setting.json
@@ -52,14 +52,14 @@
         "propertyName": null,
         "defaultValue": "1024m",
         "description": "Memory for JobManager, e.g. 1024m",
-        "type": "number"
+        "type": "text"
       },
       "taskmanager.memory.process.size": {
         "envName": null,
         "propertyName": null,
         "defaultValue": "1024m",
         "description": "Memory for TaskManager, e.g. 1024m",
-        "type": "number"
+        "type": "text"
       },
       "taskmanager.numberOfTaskSlots": {
         "envName": null,
@@ -92,7 +92,7 @@
       "zeppelin.flink.uiWebUrl": {
         "envName": null,
         "propertyName": null,
-        "defaultValue": false,
+        "defaultValue": "",
         "description": "User specified Flink JobManager url, it could be used 
in remote mode where Flink cluster is already started, or could be used as url 
template, e.g. 
https://knox-server:8443/gateway/cluster-topo/yarn/proxy/{{applicationId}}/ 
where {{applicationId}} would be replaced with yarn app id",
         "type": "string"
       },

Reply via email to