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 tu...@apache.org on 2013/07/11 02:10:54 UTC
svn commit: r1502066 -
/hadoop/common/branches/branch-2.1-beta/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/net/TestNetworkTopology.java
Author: tucu
Date: Thu Jul 11 00:10:54 2013
New Revision: 1502066
URL: http://svn.apache.org/r1502066
Log:
HADOOP-9673. NetworkTopology: when a node can't be added, print out its location for diagnostic purposes. (Colin Patrick McCabe)
Modified:
hadoop/common/branches/branch-2.1-beta/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/net/TestNetworkTopology.java
Modified: hadoop/common/branches/branch-2.1-beta/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.1-beta/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/net/TestNetworkTopology.java?rev=1502066&r1=1502065&r2=1502066&view=diff
==============================================================================
--- hadoop/common/branches/branch-2.1-beta/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/net/TestNetworkTopology.java (original)
+++ hadoop/common/branches/branch-2.1-beta/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/net/TestNetworkTopology.java Thu Jul 11 00:10:54 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."));
}
}