[zeppelin] branch branch-0.9 updated: [ZEPPELIN-5365] displayedJMWebUrl of flink interpreter is not correctly initialized

[zj...@apache.org]

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

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


The following commit(s) were added to refs/heads/branch-0.9 by this push:
     new 5133a09  [ZEPPELIN-5365] displayedJMWebUrl of flink interpreter is not correctly initialized
5133a09 is described below

commit 5133a092be63eacef4de3d3933ed5d8b2b5b8050
Author: Jeff Zhang <zj...@apache.org>
AuthorDate: Wed May 12 11:32:44 2021 +0800

    [ZEPPELIN-5365] displayedJMWebUrl of flink interpreter is not correctly initialized
    
    ### What is this PR for?
    
    Trivial PR fix the displayedJMWebUrl initialization.
    
    ### What type of PR is it?
    [Bug Fix ]
    
    ### Todos
    * [ ] - Task
    
    ### What is the Jira issue?
    * https://issues.apache.org/jira/browse/ZEPPELIN-5365
    
    ### How should this be tested?
    * CI pass and manually tested
    
    ### 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 <zj...@apache.org>
    
    Closes #4114 from zjffdu/ZEPPELIN-5365 and squashes the following commits:
    
    6de340acf7 [Jeff Zhang] [ZEPPELIN-5365] displayedJMWebUrl of flink interpreter is not correctly initialized
    
    (cherry picked from commit 160c36b599faefcbfacd430170470faeb653d988)
    Signed-off-by: Jeff Zhang <zj...@apache.org>
---
 .../main/scala/org/apache/zeppelin/flink/FlinkScalaInterpreter.scala    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/flink/interpreter/src/main/scala/org/apache/zeppelin/flink/FlinkScalaInterpreter.scala b/flink/interpreter/src/main/scala/org/apache/zeppelin/flink/FlinkScalaInterpreter.scala
index 1214009..459ff7b 100644
--- a/flink/interpreter/src/main/scala/org/apache/zeppelin/flink/FlinkScalaInterpreter.scala
+++ b/flink/interpreter/src/main/scala/org/apache/zeppelin/flink/FlinkScalaInterpreter.scala
@@ -321,7 +321,7 @@ class FlinkScalaInterpreter(val properties: Properties) {
           }
       }
 
-      if (this.displayedJMWebUrl != null) {
+      if (this.displayedJMWebUrl == null) {
         // use jmWebUrl as displayedJMWebUrl if it is not set
         this.displayedJMWebUrl = this.jmWebUrl
       }