mnmhouse opened a new pull request #4115:
URL: https://github.com/apache/zeppelin/pull/4115


   ZEPPELIN-5349.: fix  InterpreterSettingManager init error  then init again 
   ### What is this PR for?
   
   This problem is because
      The loadFromFile() method of the InterpreterSettingManager class during 
initialization
       (This method is mainly to load all interpreters and dependency 
processing) It is found that newly added  interpreter is configured with 
dependent jar packages 
         The InterpreterSetting class will start the thread to do interpreter 
dependency processing, and then trigger the 
interpreterSettingManager.saveToFile() operation. Save the interpreter.json 
file. This operation is time-consuming. At the same time, the main thread then 
executes saveToFile();  at this time there is a thread synchronization problem
         There is a high probability that the main thread exception  and after 
this happened 
          Notebook notebook = ServiceLocatorUtilities.getService(
                sharedServiceLocator, Notebook.class.getName()); will be 
executed again, causing   InterpreterSettingManager int 
    ,Thrift Server in interpreterSettingManager to start twice 
   
   
   ### What type of PR is it?
   [Bug Fix} 
   
   ### Todos
   * [ ] - Task
   
   ### What is the Jira issue?
   
   https://issues.apache.org/jira/browse/ZEPPELIN-5349
   
   ### How should this be tested?
     add a new Interpreter  
   
   
   ### Screenshots (if appropriate)
   
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to