You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zeppelin.apache.org by benoyantony <gi...@git.apache.org> on 2017/04/28 06:02:22 UTC

[GitHub] zeppelin pull request #2295: [ZEPPELIN-2463] Avoid Locking interpreterSettin...

GitHub user benoyantony opened a pull request:

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

    [ZEPPELIN-2463] Avoid Locking interpreterSettings during Notebook deletion

    ### What is this PR for?
    Deletion of a notebook requires locking interpreterSettings. If the deletion is delayed , then lock is not released. At that point, we cannot run any notebook because everything is waiting to lock interpreterSettings.
    Looking at the code, there is no reason to lock the InterpreterSettings object in InterpreterSettingManager.removeNoteInterpreterSettingBinding.
    Similarly in InterpreterSettingManager.getNoteInterpreterSettingBinding only interpreterSettingBinding is accessed and its already a thread safe object. So we can remove synchronization on InterpreterSettings
    
    ### What type of PR is it?
    Bug Fix
    
    ### What is the Jira issue?
    https://issues.apache.org/jira/browse/ZEPPELIN-2463
    
    ### How should this be tested?
    Being a concurrency issue, it is difficult to test this.
    
    ### 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/benoyantony/zeppelin ZEPPELIN-2463

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

    https://github.com/apache/zeppelin/pull/2295.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 #2295
    
----
commit 5fbe6d4cbd9cbf0581e5d303cd5e4005268c2c2a
Author: Benoy Antony <be...@apache.org>
Date:   2017-04-28T05:58:09Z

    ZEPPELIN-2463 Avoid Locking interpreterSettings during Notebook deletion

----


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin pull request #2295: [ZEPPELIN-2463] Avoid Locking interpreterSettin...

Posted by benoyantony <gi...@git.apache.org>.
Github user benoyantony closed the pull request at:

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


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---