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 ay...@apache.org on 2019/10/20 07:06:10 UTC
[hadoop] branch branch-2 updated: HADOOP-16662. Remove unnecessary
InnerNode check in NetworkTopology#add(). Contributed by Lisheng Sun.
This is an automated email from the ASF dual-hosted git repository.
ayushsaxena pushed a commit to branch branch-2
in repository https://gitbox.apache.org/repos/asf/hadoop.git
The following commit(s) were added to refs/heads/branch-2 by this push:
new c78eb00 HADOOP-16662. Remove unnecessary InnerNode check in NetworkTopology#add(). Contributed by Lisheng Sun.
c78eb00 is described below
commit c78eb006f909f5e7c545e492488ac5a9f01217fb
Author: Ayush Saxena <ay...@apache.org>
AuthorDate: Sat Oct 19 11:05:54 2019 +0530
HADOOP-16662. Remove unnecessary InnerNode check in NetworkTopology#add(). Contributed by Lisheng Sun.
---
.../src/main/java/org/apache/hadoop/net/NetworkTopology.java | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/net/NetworkTopology.java b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/net/NetworkTopology.java
index 1f077a7..52cf420 100644
--- a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/net/NetworkTopology.java
+++ b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/net/NetworkTopology.java
@@ -146,10 +146,8 @@ public class NetworkTopology {
if (rack == null) {
incrementRacks();
}
- if (!(node instanceof InnerNode)) {
- if (depthOfAllLeaves == -1) {
- depthOfAllLeaves = node.getLevel();
- }
+ if (depthOfAllLeaves == -1) {
+ depthOfAllLeaves = node.getLevel();
}
}
LOG.debug("NetworkTopology became:\n{}", this);
---------------------------------------------------------------------
To unsubscribe, e-mail: common-commits-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-commits-help@hadoop.apache.org