svn commit: r954988 - /cassandra/trunk/src/java/org/apache/cassandra/locator/RackAwareStrategy.java

[jb...@apache.org]

Author: jbellis
Date: Tue Jun 15 18:02:53 2010
New Revision: 954988

URL: http://svn.apache.org/viewvc?rev=954988&view=rev
Log:
update RAS javadoc to clarify that only the 3rd replica actually cares about rack placement.  patch by jbellis

Modified:
    cassandra/trunk/src/java/org/apache/cassandra/locator/RackAwareStrategy.java

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=954988&r1=954987&r2=954988&view=diff
==============================================================================
--- cassandra/trunk/src/java/org/apache/cassandra/locator/RackAwareStrategy.java (original)
+++ cassandra/trunk/src/java/org/apache/cassandra/locator/RackAwareStrategy.java Tue Jun 15 18:02:53 2010
@@ -25,12 +25,12 @@ import java.util.*;
 import org.apache.cassandra.config.DatabaseDescriptor;
 import org.apache.cassandra.dht.Token;
 
-/*
+/**
  * This Replication Strategy returns the nodes responsible for a given
  * key but respects rack awareness. It places one replica in a
  * different data center from the first (if there is any such data center),
- * and remaining replicas in different racks in the same datacenter as
- * the first.
+ * the third replica in a different rack in the first datacenter, and
+ * any remaining replicas on the first unused nodes on the ring.
  */
 public class RackAwareStrategy extends AbstractReplicationStrategy
 {