GitHub user hammertank opened a pull request:

    https://github.com/apache/zeppelin/pull/2319

    [ZEPPELIN-2437] Executions of notebooks are blocked after running 50 
different notebooks under SparkInterpreter scoped mode

    ### What is this PR for?
    Removes Scheduler when RemoteInterpreterServer close an interpreter.
    
    ### What type of PR is it?
    [Bug Fix]
    
    ### Todos
    * [x] - Removes Scheduler when calling RemoteInterpreterServer.close()
    
    ### What is the Jira issue?
    * https://issues.apache.org/jira/browse/ZEPPELIN-2437
    
    ### How should this be tested?
    RemoteInterpreterServerTest#testClose(TBD)
    
    ### Questions:
    * Does the licenses files need update? No
    * Is there breaking changes for older versions? No
    * Does this needs documentation? No


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/hammertank/zeppelin ZEPPELIN-2437

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/zeppelin/pull/2319.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #2319
    
----
commit d5a3ffa94e56d45534caedf71e7adfd7b70c8bf4
Author: Jeff Zhang <zjf...@apache.org>
Date:   2017-03-22T07:19:42Z

    [HOTFIX][ZEPPELIN-2294]. Interpreter fail exception is not propagated to 
frontend
    
    ### What is this PR for?
    
    This PR would address the issue of Interpreter fail exception is not 
propagated to frontend. This is due to the bug fix of restarting button issue.  
Make referenceCount as number of sessions attached to this process. Open and 
close all the interpreters in one session together. I don't think there's 
scenario for close a single interpreter for now.
    
    ### What type of PR is it?
    [Bug Fix | Improvement]
    
    ### Todos
    * [ ] - Task
    
    ### What is the Jira issue?
    * https://issues.apache.org/jira/browse/ZEPPELIN-2294
    
    ### How should this be tested?
    I don't have time to write test, just verify zombie process issue manually 
in the following scenario
    * Per User + Isolated
    * Per Note + Isolated
    * Per User + Scoped
    * Per Note + Scoped
    
    ### Screenshots (if appropriate)
    Before
    
![2017-03-22_1538](https://cloud.githubusercontent.com/assets/164491/24198052/227d42e6-0f3f-11e7-9918-bf9827e44f92.png)
    
    After
    
![2017-03-22_1523](https://cloud.githubusercontent.com/assets/164491/24198062/31043cc0-0f3f-11e7-8ab3-87938e3918ce.png)
    
    ### 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 #2175 from zjffdu/ZEPPELIN-2294 and squashes the following commits:
    
    a3f8aa0b [Jeff Zhang] [ZEPPELIN-2294]. Interpreter fail exception is not 
propagated to frontend
    
    (cherry picked from commit 0aea2416dc60fb6e2e181ee1c593d9d2c575bcb7)
    Signed-off-by: Jongyoul Lee <jongy...@apache.org>
    
    (cherry picked from commit ddf2378)

commit 4a4aea3686d2e0445f066127db9f95c300cece7e
Author: z0621 <zhouyf0...@dtdream.com>
Date:   2017-05-04T07:12:04Z

    Remove Scheduler when Interpreter is closed

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to