Repository: hadoop Updated Branches: refs/heads/branch-2 8d69b7f2d -> d69b7358b
YARN-7647. NM print inappropriate error log when node-labels is enabled. Contributed by Yang Wang. (cherry picked from commit 7efc4f76885348730728c0201dd0d1a89b213e9c) Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/d69b7358 Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/d69b7358 Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/d69b7358 Branch: refs/heads/branch-2 Commit: d69b7358b65128197b4c0fe5ef3c02f3d59864b3 Parents: 8d69b7f Author: Weiwei Yang <[email protected]> Authored: Wed Dec 13 13:11:41 2017 +0800 Committer: Weiwei Yang <[email protected]> Committed: Wed Dec 13 13:28:48 2017 +0800 ---------------------------------------------------------------------- .../yarn/server/nodemanager/NodeStatusUpdaterImpl.java | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/hadoop/blob/d69b7358/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/NodeStatusUpdaterImpl.java ---------------------------------------------------------------------- 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 65bde63..b4356cb 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 @@ -1208,9 +1208,12 @@ public class NodeStatusUpdaterImpl extends AbstractService implements public void verifyRMHeartbeatResponseForNodeLabels( NodeHeartbeatResponse response) { if (areLabelsSentToRM) { - if (response.getAreNodeLabelsAcceptedByRM() && LOG.isDebugEnabled()) { - LOG.debug("Node Labels {" + StringUtils.join(",", previousNodeLabels) - + "} were Accepted by RM "); + if (response.getAreNodeLabelsAcceptedByRM()) { + if(LOG.isDebugEnabled()){ + LOG.debug( + "Node Labels {" + StringUtils.join(",", previousNodeLabels) + + "} were Accepted by RM "); + } } else { // case where updated labels from NodeLabelsProvider is sent to RM and // RM rejected the labels --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
