[YARN-6329] Remove unnecessary TODO comment from AppLogAggregatorImpl.java

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 2.8.0
• Fix Version/s: 2.9.0, 3.0.0-alpha4
• Labels: newbie
• Hadoop Flags: Reviewed
• Assignee: victor bertschinger
• Reporter: Akira Ajisaka
• Created: 13/Mar/17 05:52
• Updated: 29/Mar/17 02:00
• Resolved: 29/Mar/17 01:47

Description

After YARN-3116, this TODO comment is unnecessary.

  // TODO: The condition: containerId.getId() == 1 to determine an AM container
  // is not always true.
  private boolean shouldUploadLogs(ContainerLogContext logContext) {
    return logAggPolicy.shouldDoLogAggregation(logContext);
  }