This is an automated email from the ASF dual-hosted git repository.
jbrennan pushed a commit to branch branch-3.2
in repository https://gitbox.apache.org/repos/asf/hadoop.git
The following commit(s) were added to refs/heads/branch-3.2 by this push:
new 1efb54b YARN-10430. Log improvements in NodeStatusUpdaterImpl.
Contributed by Bilwa S T.
1efb54b is described below
commit 1efb54bd524d6961511712b43d3883c9e32a2b2a
Author: Jim Brennan <[email protected]>
AuthorDate: Tue Sep 15 16:27:08 2020 +0000
YARN-10430. Log improvements in NodeStatusUpdaterImpl. Contributed by Bilwa
S T.
---
.../server/nodemanager/NodeStatusUpdaterImpl.java | 20 ++++++++++----------
1 file changed, 10 insertions(+), 10 deletions(-)
diff --git
a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/NodeStatusUpdaterImpl.java
b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/NodeStatusUpdaterImpl.java
index 6bffb79..682937d 100644
---
a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/NodeStatusUpdaterImpl.java
+++
b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/NodeStatusUpdaterImpl.java
@@ -403,12 +403,10 @@ public class NodeStatusUpdaterImpl extends
AbstractService implements
List<LogAggregationReport> logAggregationReports =
context.getNMLogAggregationStatusTracker()
.pullCachedLogAggregationReports();
- if (LOG.isDebugEnabled()) {
- 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);
}
}
@@ -621,11 +619,11 @@ public class NodeStatusUpdaterImpl extends
AbstractService implements
}
containerStatuses.addAll(pendingCompletedContainers.values());
-
- if (LOG.isDebugEnabled()) {
- LOG.debug("Sending out " + containerStatuses.size()
- + " container statuses: " + containerStatuses);
+ if (!containerStatuses.isEmpty()) {
+ LOG.debug("Sending out {} container statuses: {}",
+ containerStatuses.size(), containerStatuses);
}
+
return containerStatuses;
}
@@ -663,8 +661,10 @@ public class NodeStatusUpdaterImpl extends AbstractService
implements
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;
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
