GitHub user jongyoul opened a pull request:
https://github.com/apache/zeppelin/pull/2935
ZEPPELIN-3411 Long running logic inside synchronized block in
InterpreterSettingManager
### What is this PR for?
Removing redundant synchronized code to avoid blocking other logics.
### What type of PR is it?
[Bug Fix]
### Todos
* [x] - Change synchronized block to read/write lock
### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-3411
### How should this be tested?
* Current tests should be passed
### Screenshots (if appropriate)
### 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/jongyoul/zeppelin ZEPPELIN-3411
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/zeppelin/pull/2935.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 #2935
----
commit 24be69298c8a3ca514b478a0ca95e9ac0f151178
Author: Jongyoul Lee <jongyoul@...>
Date: 2018-04-17T09:38:56Z
Removed all synchronized blocks and replace them to read/write lock
----
---
