You are viewing a plain text version of this content. The canonical link for it is here.
Posted to hdfs-commits@hadoop.apache.org by cm...@apache.org on 2013/06/27 03:40:06 UTC
svn commit: r1497180 -
/hadoop/common/branches/branch-2/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/net/TestNetworkTopology.java
Author: cmccabe
Date: Thu Jun 27 01:40:05 2013
New Revision: 1497180
URL: http://svn.apache.org/r1497180
Log:
HADOOP-9673. NetworkTopology: When a node cannot be added, print out its location for diagnostic purposes
Modified:
hadoop/common/branches/branch-2/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/net/TestNetworkTopology.java
Modified: hadoop/common/branches/branch-2/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/net/TestNetworkTopology.java
URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-2/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/net/TestNetworkTopology.java?rev=1497180&r1=1497179&r2=1497180&view=diff
==============================================================================
--- hadoop/common/branches/branch-2/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/net/TestNetworkTopology.java (original)
+++ hadoop/common/branches/branch-2/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/net/TestNetworkTopology.java Thu Jun 27 01:40:05 2013
@@ -91,9 +91,10 @@ public class TestNetworkTopology {
invalCluster.add(invalDataNodes[2]);
fail("expected InvalidTopologyException");
} catch (NetworkTopology.InvalidTopologyException e) {
- assertEquals(e.getMessage(), "Invalid network topology. " +
+ assertTrue(e.getMessage().startsWith("Failed to add "));
+ assertTrue(e.getMessage().contains(
"You cannot have a rack and a non-rack node at the same " +
- "level of the network topology.");
+ "level of the network topology."));
}
}