[YARN-10430] Log improvements in NodeStatusUpdaterImpl

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 3.2.2, 3.4.0, 3.3.1, 3.2.3
• Component/s: nodemanager
• Assignee: Bilwa S T
• Reporter: Bilwa S T
• Created: 09/Sep/20 08:21
• Updated: 10/Jun/21 08:15
• Resolved: 14/Sep/20 21:39

Description

I think in below places log should be printed only if list size is not zero.

if (LOG.isDebugEnabled()) {
          LOG.debug("The cache log aggregation status size:"
              + logAggregationReports.size());
 }

LOG.info("Sending out " + containerStatuses.size()
      + " NM container statuses: " + containerStatuses);

if (LOG.isDebugEnabled()) {
      LOG.debug("Sending out " + containerStatuses.size()
          + " container statuses: " + containerStatuses);
}