Author: jbellis
Date: Tue Apr 12 15:17:52 2011
New Revision: 1091455

URL: http://svn.apache.org/viewvc?rev=1091455&view=rev
Log:
cli no longer divides read_repair_chance by 100
patch by Aaron Morton; reviewed by jbellis for CASSANDRA-2458

Modified:
    cassandra/branches/cassandra-0.8/CHANGES.txt
    
cassandra/branches/cassandra-0.8/src/java/org/apache/cassandra/cli/CliClient.java

Modified: cassandra/branches/cassandra-0.8/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/cassandra/branches/cassandra-0.8/CHANGES.txt?rev=1091455&r1=1091454&r2=1091455&view=diff
==============================================================================
--- cassandra/branches/cassandra-0.8/CHANGES.txt (original)
+++ cassandra/branches/cassandra-0.8/CHANGES.txt Tue Apr 12 15:17:52 2011
@@ -22,6 +22,7 @@
  * remove "nodetool loadbalance" (CASSANDRA-2448)
  * multithreaded compaction (CASSANDRA-2191)
  * compaction throttling (CASSANDRA-2156)
+ * cli no longer divides read_repair_chance by 100 (CASSANDRA-2458)
 
 
 0.7.5

Modified: 
cassandra/branches/cassandra-0.8/src/java/org/apache/cassandra/cli/CliClient.java
URL: 
http://svn.apache.org/viewvc/cassandra/branches/cassandra-0.8/src/java/org/apache/cassandra/cli/CliClient.java?rev=1091455&r1=1091454&r2=1091455&view=diff
==============================================================================
--- 
cassandra/branches/cassandra-0.8/src/java/org/apache/cassandra/cli/CliClient.java
 (original)
+++ 
cassandra/branches/cassandra-0.8/src/java/org/apache/cassandra/cli/CliClient.java
 Tue Apr 12 15:17:52 2011
@@ -1039,10 +1039,10 @@ public class CliClient extends CliUserHe
                 cfDef.setKey_cache_size(Double.parseDouble(mValue));
                 break;
             case READ_REPAIR_CHANCE:
-                double chance = Double.parseDouble(mValue) / 100;
+                double chance = Double.parseDouble(mValue);
 
-                if (chance > 1)
-                    throw new RuntimeException("Error: read_repair_chance / 
100 should not be greater than 1.");
+                if (chance < 0 || chance > 1)
+                    throw new RuntimeException("Error: read_repair_chance must 
be between 0 and 1.");
 
                 cfDef.setRead_repair_chance(chance);
                 break;


Reply via email to