[HADOOP-18065] ExecutorHelper.logThrowableFromAfterExecute() is too noisy.

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 3.3.1
• Fix Version/s: 3.3.2
• Labels: pull-request-available
• Target Version/s: 3.3.5
• Assignee: Mukund Thakur
• Reporter: Mukund Thakur
• Created: 05/Jan/22 09:16
• Updated: 19/Jan/22 17:48
• Resolved: 06/Jan/22 09:58

Description

if (t == null && r instanceof Future<?> && ((Future<?>) r).isDone()) {
  try {
    ((Future<?>) r).get();
  } catch (ExecutionException ee) {
    LOG.warn(
        "Execution exception when running task in " + Thread.currentThread()
            .getName());
    t = ee.getCause();
  } catch (InterruptedException ie) {
    LOG.warn("Thread (" + Thread.currentThread() + ") interrupted: ", ie);
    Thread.currentThread().interrupt();
  } catch (Throwable throwable) {
    t = throwable;
  }
}

if (t != null) {
  LOG.warn("Caught exception in thread " + Thread
      .currentThread().getName() + ": ", t);
} 

We should downgrade the logging here from warn to debug.

 

CC [e-mail]  mehakmeetSingh 

Issue Links

• links to: GitHub Pull Request #3860