Repository: zeppelin
Updated Branches:
  refs/heads/master f28e96317 -> 7954d8071


ZEPPELIN-3794. NPE when running paragraph without interpreter specified

### What is this PR for?
This issue happens for the old note where no defaultInterpreterGroup is 
specified in note.json.

### What type of PR is it?
[Bug Fix]

### Todos
* [ ] - Task

### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-3794

### 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 #3191 from zjffdu/ZEPPELIN-3794 and squashes the following commits:

a8a469fa0 [Jeff Zhang] ZEPPELIN-3794. NPE when running paragraph without 
interpreter specified


Project: http://git-wip-us.apache.org/repos/asf/zeppelin/repo
Commit: http://git-wip-us.apache.org/repos/asf/zeppelin/commit/7954d807
Tree: http://git-wip-us.apache.org/repos/asf/zeppelin/tree/7954d807
Diff: http://git-wip-us.apache.org/repos/asf/zeppelin/diff/7954d807

Branch: refs/heads/master
Commit: 7954d8071f2b1c46259c449983825f26afce6d6d
Parents: f28e963
Author: Jeff Zhang <zjf...@apache.org>
Authored: Thu Sep 27 11:29:43 2018 +0800
Committer: Jeff Zhang <zjf...@apache.org>
Committed: Fri Sep 28 08:53:51 2018 +0800

----------------------------------------------------------------------
 .../src/main/java/org/apache/zeppelin/notebook/Note.java         | 4 ++++
 1 file changed, 4 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/zeppelin/blob/7954d807/zeppelin-zengine/src/main/java/org/apache/zeppelin/notebook/Note.java
----------------------------------------------------------------------
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 8f916e1..74b08b2 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
@@ -195,6 +195,10 @@ public class Note implements JsonSerializable {
   }
 
   public String getDefaultInterpreterGroup() {
+    if (defaultInterpreterGroup == null) {
+      defaultInterpreterGroup = ZeppelinConfiguration.create()
+          
.getString(ZeppelinConfiguration.ConfVars.ZEPPELIN_INTERPRETER_GROUP_DEFAULT);
+    }
     return defaultInterpreterGroup;
   }
 

Reply via email to