[zeppelin] branch branch-0.9 updated: [hotfix] NPE in Note#getBindedInterpreterSettings

[zjffdu]

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 1c878c3  [hotfix] NPE in Note#getBindedInterpreterSettings
1c878c3 is described below

commit 1c878c3d8f864a1b124fd3d1537f79d641f3bf2e
Author: Jeff Zhang <zjf...@apache.org>
AuthorDate: Mon Jun 29 15:04:58 2020 +0800

    [hotfix] NPE in Note#getBindedInterpreterSettings
---
 .../src/main/java/org/apache/zeppelin/notebook/Note.java       | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git 
a/zeppelin-zengine/src/main/java/org/apache/zeppelin/notebook/Note.java 
b/zeppelin-zengine/src/main/java/org/apache/zeppelin/notebook/Note.java
index ad353a8..90a2d3a 100644
--- a/zeppelin-zengine/src/main/java/org/apache/zeppelin/notebook/Note.java
+++ b/zeppelin-zengine/src/main/java/org/apache/zeppelin/notebook/Note.java
@@ -985,10 +985,12 @@ public class Note implements JsonSerializable {
       settings.add(defaultIntpSetting);
     }
     // add the interpreter setting with the same group of default interpreter 
group
-    for (InterpreterSetting intpSetting : interpreterSettingManager.get()) {
-      if (intpSetting.getGroup().equals(defaultIntpSetting.getGroup())) {
-        if (intpSetting.isUserAuthorized(userAndRoles)) {
-          settings.add(intpSetting);
+    if (defaultIntpSetting != null) {
+      for (InterpreterSetting intpSetting : interpreterSettingManager.get()) {
+        if (intpSetting.getGroup().equals(defaultIntpSetting.getGroup())) {
+          if (intpSetting.isUserAuthorized(userAndRoles)) {
+            settings.add(intpSetting);
+          }
         }
       }
     }