You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by jb...@apache.org on 2009/05/12 23:26:43 UTC
svn commit: r774097 -
/incubator/cassandra/trunk/test/unit/org/apache/cassandra/locator/RackUnawareStrategyTest.java
Author: jbellis
Date: Tue May 12 21:26:42 2009
New Revision: 774097
URL: http://svn.apache.org/viewvc?rev=774097&view=rev
Log:
fix RackUnawareStrategyTest -- endpoint asserts 'host' is an ip address (to make sure we're not mixing hostnames in again) so create a suitable fake IP for the test. patch by jbellis for CASSANDRA-163
Modified:
incubator/cassandra/trunk/test/unit/org/apache/cassandra/locator/RackUnawareStrategyTest.java
Modified: incubator/cassandra/trunk/test/unit/org/apache/cassandra/locator/RackUnawareStrategyTest.java
URL: http://svn.apache.org/viewvc/incubator/cassandra/trunk/test/unit/org/apache/cassandra/locator/RackUnawareStrategyTest.java?rev=774097&r1=774096&r2=774097&view=diff
==============================================================================
--- incubator/cassandra/trunk/test/unit/org/apache/cassandra/locator/RackUnawareStrategyTest.java (original)
+++ incubator/cassandra/trunk/test/unit/org/apache/cassandra/locator/RackUnawareStrategyTest.java Tue May 12 21:26:42 2009
@@ -14,8 +14,6 @@
public class RackUnawareStrategyTest
{
- // TODO fix these
- /*
@Test
public void testBigIntegerStorageEndPoints()
{
@@ -55,7 +53,7 @@
List<EndPoint> hosts = new ArrayList<EndPoint>();
for (int i = 0; i < endPointTokens.length; i++)
{
- EndPoint ep = new EndPoint(String.valueOf(i), 7001);
+ EndPoint ep = new EndPoint("127.0.0." + String.valueOf(i + 1), 7001);
tmd.update(endPointTokens[i], ep);
hosts.add(ep);
}
@@ -70,5 +68,4 @@
}
}
}
- */
}