[PR] [FIX] Fix IllegalThreadStateException in ComponentStarter shutdown hook [bookkeeper]

["merlimat (via GitHub)" <gi...@apache.org>]

merlimat opened a new pull request, #4733:
URL: https://github.com/apache/bookkeeper/pull/4733

   ## Summary
   - The `UncaughtExceptionHandler` in `ComponentStarter` calls `shutdownHookThread.start()`, which throws `IllegalThreadStateException` if the thread was already started (by a prior exception or the JVM shutdown sequence)
   - This causes the JVM to print `Exception: java.lang.IllegalThreadStateException thrown from the UncaughtExceptionHandler in thread "BookieDeathWatcher-<port>"`
   - Fix: catch `IllegalThreadStateException` since it simply means shutdown is already in progress
   
   ## Test plan
   - [ ] Verify the exception no longer appears when shutting down a bookie server
   - [ ] Existing unit tests pass


-- 
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: commits-unsubscribe@bookkeeper.apache.org

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

Re: [PR] [FIX] Fix IllegalThreadStateException in ComponentStarter shutdown hook [bookkeeper]

["merlimat (via GitHub)" <gi...@apache.org>]

merlimat merged PR #4733:
URL: https://github.com/apache/bookkeeper/pull/4733


-- 
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: commits-unsubscribe@bookkeeper.apache.org

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