zeppelin icon indicating copy to clipboard operation
zeppelin copied to clipboard

[ZEPPELIN-6237] Add serviceLocator shutdown when server is close

Open proceane opened this issue 5 months ago • 1 comments

What is this PR for?

In the zeppelin-server/rest module, it was observed that when running multiple tests together, configuration values of notebooks used in previous tests remain. To address this, code has been added to shutdown serviceLocator when the test finishes and the server is stopped.

What type of PR is it?

Bug Fix

Todos

  • Add serviceLocatior shutdown

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-6237

How should this be tested?

  • In the zeppelin-server/rest module, run test.

Screenshots (if appropriate)

Questions:

  • Does the license files need to update? No
  • Is there breaking changes for older versions? No
  • Does this needs documentation? No

proceane avatar Jul 13 '25 16:07 proceane

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.

Reamer avatar Jul 14 '25 07:07 Reamer