[HADOOP-2323] JobTracker.close() prints stack traces for exceptions that are not errors

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 0.16.0
• Fix Version/s: 0.16.0
• Assignee: Jim Kellerman
• Reporter: Jim Kellerman
• Created: 30/Nov/07 22:46
• Updated: 08/Jul/09 16:52
• Resolved: 15/Dec/07 00:23

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!