Reamer commented on PR #4965:
URL: https://github.com/apache/zeppelin/pull/4965#issuecomment-3068087049

   This pull request is a workaround for me, because the ServiceLocator should 
actually be executed when Zeppelin is shut down.
   
https://github.com/apache/zeppelin/blob/d7740440d1992606c577e1e0d5f884d3d4bb3286/zeppelin-server/src/main/java/org/apache/zeppelin/server/ZeppelinServer.java#L357-L377
   
   Somewhere here should be `sharedServiceLocator.shutdown();`.
   
   Unfortunately, my efforts failed because of the recovery tests.
   Would you like to try a real shutdown here?
   
   In my opinion, the `close()` method of the “services” must take the recovery 
mechanism into account and not at the top of the Zeppelin server.


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to