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 2010/08/03 17:49:15 UTC
svn commit: r981940 - in
/cassandra/trunk/src/java/org/apache/cassandra/locator:
DatacenterShardStrategy.java RackAwareStrategy.java
Author: jbellis
Date: Tue Aug 3 15:49:15 2010
New Revision: 981940
URL: http://svn.apache.org/viewvc?rev=981940&view=rev
Log:
remove instanceof checks for AbstractRackAwareSnitch. patch by Sylvain Lebresne; reviewed by jbellis for CASSANDRA-1349
Modified:
cassandra/trunk/src/java/org/apache/cassandra/locator/DatacenterShardStrategy.java
cassandra/trunk/src/java/org/apache/cassandra/locator/RackAwareStrategy.java
Modified: cassandra/trunk/src/java/org/apache/cassandra/locator/DatacenterShardStrategy.java
URL: http://svn.apache.org/viewvc/cassandra/trunk/src/java/org/apache/cassandra/locator/DatacenterShardStrategy.java?rev=981940&r1=981939&r2=981940&view=diff
==============================================================================
--- cassandra/trunk/src/java/org/apache/cassandra/locator/DatacenterShardStrategy.java (original)
+++ cassandra/trunk/src/java/org/apache/cassandra/locator/DatacenterShardStrategy.java Tue Aug 3 15:49:15 2010
@@ -58,8 +58,6 @@ public class DatacenterShardStrategy ext
public DatacenterShardStrategy(String table, TokenMetadata tokenMetadata, IEndpointSnitch snitch, Map<String, String> configOptions) throws ConfigurationException
{
super(table, tokenMetadata, snitch, configOptions);
- if ((!(snitch instanceof AbstractRackAwareSnitch)))
- throw new IllegalArgumentException("DatacenterShardStrategy requires a rack-aware endpointsnitch");
this.snitch = (AbstractRackAwareSnitch)snitch;
Map<String, Integer> newDatacenters = new HashMap<String, Integer>();
Modified: cassandra/trunk/src/java/org/apache/cassandra/locator/RackAwareStrategy.java
URL: http://svn.apache.org/viewvc/cassandra/trunk/src/java/org/apache/cassandra/locator/RackAwareStrategy.java?rev=981940&r1=981939&r2=981940&view=diff
==============================================================================
--- cassandra/trunk/src/java/org/apache/cassandra/locator/RackAwareStrategy.java (original)
+++ cassandra/trunk/src/java/org/apache/cassandra/locator/RackAwareStrategy.java Tue Aug 3 15:49:15 2010
@@ -37,8 +37,6 @@ public class RackAwareStrategy extends A
public RackAwareStrategy(String table, TokenMetadata tokenMetadata, IEndpointSnitch snitch, Map<String, String> configOptions)
{
super(table, tokenMetadata, snitch, configOptions);
- if (!(snitch instanceof AbstractRackAwareSnitch))
- throw new IllegalArgumentException(("RackAwareStrategy requires AbstractRackAwareSnitch."));
}
public Set<InetAddress> calculateNaturalEndpoints(Token token, TokenMetadata metadata)