Details
-
Bug
-
Status: Closed
-
Minor
-
Resolution: Fixed
-
0.16.0
-
None
-
None
Description
JobTracker.close() prints a stack trace for an interrupted exception even though it was the method that interrupted the thread that threw the exception. For example:
this.expireTrackers.stopTracker(); try { this.expireTrackersThread.interrupt(); this.expireTrackersThread.join(); } catch (InterruptedException ex) { ex.printStackTrace(); }
Well of course it is going to catch an InterruptedException after it just interrupted the thread!
This is not an error and should not be dumped to the logs!
In other circumstances, catching InterruptedException is entirely appropriate. Just not in close where you've told the thread to shutdown and then interrupted it to ensure it does!