YARN-10430. Log improvements in NodeStatusUpdaterImpl. Contributed by Bilwa S T.

This commit is contained in:
Jim Brennan 2020-09-14 21:19:14 +00:00
parent 6d3312de47
commit 90894ea641

View File

@ -406,10 +406,10 @@ nodeManagerVersionId, containerReports, getRunningApplications(),
List<LogAggregationReport> logAggregationReports =
context.getNMLogAggregationStatusTracker()
.pullCachedLogAggregationReports();
LOG.debug("The cache log aggregation status size:{}",
logAggregationReports.size());
if (logAggregationReports != null
&& !logAggregationReports.isEmpty()) {
LOG.debug("The cache log aggregation status size:{}",
logAggregationReports.size());
request.setLogAggregationReportsForApps(logAggregationReports);
}
}
@ -624,8 +624,10 @@ protected List<ContainerStatus> getContainerStatuses() throws IOException {
}
containerStatuses.addAll(pendingCompletedContainers.values());
LOG.debug("Sending out {} container statuses: {}",
containerStatuses.size(), containerStatuses);
if (!containerStatuses.isEmpty()) {
LOG.debug("Sending out {} container statuses: {}",
containerStatuses.size(), containerStatuses);
}
return containerStatuses;
}
@ -664,8 +666,10 @@ private List<NMContainerStatus> getNMContainerStatuses() throws IOException {
addCompletedContainer(containerId);
}
}
LOG.info("Sending out " + containerStatuses.size()
+ " NM container statuses: " + containerStatuses);
if (!containerStatuses.isEmpty()) {
LOG.info("Sending out " + containerStatuses.size()
+ " NM container statuses: " + containerStatuses);
}
return containerStatuses;
}