jmxtrans icon indicating copy to clipboard operation
jmxtrans copied to clipboard

Handle errors on reload with the default uncaught exception handler

Open afn opened this issue 7 years ago • 2 comments

This code (added in #711) was throwing a RuntimeException, which I mistakenly believed would invoke the default uncaught exception handler. Instead, ThreadPoolExecutor swallows the RuntimeException and terminates the executor.

Calling the default uncaught exception handler will currently just log the exception to stderr. Once #709 is merged, it will trigger our global uncaught exception handler, which logs the error and shuts down jmxtrans.

afn avatar Oct 11 '18 19:10 afn

Rebased.

afn avatar Nov 01 '18 19:11 afn

Why a separate PR from #709 ?

gquintana avatar Dec 03 '18 20:12 gquintana