Elgato_dark/zeppelin
1
0
Fork 0
mirror of https://github.com/apache/zeppelin synced 2026-05-24 09:38:26 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

Don't destroy interpreter on note memove or change bindings when interpreter.option.perNoteSession is false

Browse source
This commit is contained in:
Lee moon soo 2016-02-07 08:12:43 +09:00
parent fc9cb3f4b5
commit 366b651943
2 changed files with 5 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
zeppelin-interpreter/src/test/java/org/apache/zeppelin/interpreter/remote/RemoteInterpreterTest.java
View file

@ -119,10 +119,10 @@ public class RemoteInterpreterTest {
assertEquals(0, process.getNumIdleClient());
assertEquals(0, process.referenceCount());
intpA.open();
intpA.open(); // initializa all interpreters in the same group
assertTrue(process.isRunning());
assertEquals(1, process.getNumIdleClient());
assertEquals(1, process.referenceCount());
assertEquals(2, process.referenceCount());
intpA.interpret("1",
new InterpreterContext(

3
zeppelin-zengine/src/main/java/org/apache/zeppelin/interpreter/InterpreterFactory.java
View file

@ -442,6 +442,9 @@ public class InterpreterFactory {
public void removeInterpretersForNote(InterpreterSetting interpreterSetting,
String noteId) {
if (!interpreterSetting.getOption().isPerNoteSession()) {
return;
}
InterpreterGroup interpreterGroup = interpreterSetting.getInterpreterGroup();
interpreterGroup.close(noteId);
interpreterGroup.destroy(noteId);