keith-turner commented on PR #5808: URL: https://github.com/apache/accumulo/pull/5808#issuecomment-3201395851
> Yeah, so we should probably just catch Throwable in both cases. I added a test concurrent to you, the following prints both, but doesn't link them using the suppression mechanism. Not completely sure, but linking them may be useful because then the uncaught ecxception handler will see an error was thrown. Catching throwable would log the error, it just would not percolate out of the main method. Using try w/ resources it does percolate out of the main method, but then it does not log. Maybe need to do both? -- 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]
