HDDS-1896. Suppress WARN log from NetworkTopology#getDistanceCost. Contributed by Xiaoyu Yao. (#1214)

xiaoyuyao · bharatviswa504 · commit 065cbc6b5460 · 2019-08-03T22:33:01.000-07:00
diff --git a/hadoop-hdds/common/src/main/java/org/apache/hadoop/hdds/scm/net/NetworkTopologyImpl.java b/hadoop-hdds/common/src/main/java/org/apache/hadoop/hdds/scm/net/NetworkTopologyImpl.java
@@ -595,7 +595,7 @@ public int getDistanceCost(Node node1, Node node2) {
     try {
       if ((node1.getAncestor(maxLevel - 1) != clusterTree) ||
           (node2.getAncestor(maxLevel - 1) != clusterTree)) {
-        LOG.warn("One of the nodes is outside of network topology");
+        LOG.debug("One of the nodes is outside of network topology");
         return Integer.MAX_VALUE;
       }
       int level1 = node1.getLevel();

Original file line number	Diff line number	Diff line change
`@@ -595,7 +595,7 @@ public int getDistanceCost(Node node1, Node node2) {`
`595`	`595`	`try {`
`596`	`596`	`if ((node1.getAncestor(maxLevel - 1) != clusterTree) \|\|`
`597`	`597`	`(node2.getAncestor(maxLevel - 1) != clusterTree)) {`
`598`		`- LOG.warn("One of the nodes is outside of network topology");`
	`598`	`+ LOG.debug("One of the nodes is outside of network topology");`
`599`	`599`	`return Integer.MAX_VALUE;`
`600`	`600`	`}`
`601`	`601`	`int level1 = node1.getLevel();`