You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by jb...@apache.org on 2020/09/14 21:29:58 UTC
[hadoop] branch branch-3.3 updated: YARN-10430. Log improvements in
NodeStatusUpdaterImpl. Contributed by Bilwa S T.
This is an automated email from the ASF dual-hosted git repository.
jbrennan pushed a commit to branch branch-3.3
in repository https://gitbox.apache.org/repos/asf/hadoop.git
The following commit(s) were added to refs/heads/branch-3.3 by this push:
new 0ec21b9 YARN-10430. Log improvements in NodeStatusUpdaterImpl. Contributed by Bilwa S T.
0ec21b9 is described below
commit 0ec21b96670e308fd152547b0f86f07b8d6d0dd0
Author: Jim Brennan <jb...@apache.org>
AuthorDate: Mon Sep 14 21:19:14 2020 +0000
YARN-10430. Log improvements in NodeStatusUpdaterImpl. Contributed by Bilwa S T.
(cherry picked from commit 90894ea641f571ebc092f177538f40157f1b3113)
---
.../yarn/server/nodemanager/NodeStatusUpdaterImpl.java | 16 ++++++++++------
1 file changed, 10 insertions(+), 6 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 5c13da2..dcfe83c 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
@@ -405,10 +405,10 @@ public class NodeStatusUpdaterImpl extends AbstractService implements
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);
}
}
@@ -623,8 +623,10 @@ public class NodeStatusUpdaterImpl extends AbstractService implements
}
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;
}
@@ -663,8 +665,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: common-commits-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-commits-help@hadoop.apache.org