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 2011/06/29 03:14:44 UTC

svn commit: r1140928 - in /cassandra/branches/cassandra-0.7: CHANGES.txt src/java/org/apache/cassandra/cli/CliClient.java

Author: jbellis
Date: Wed Jun 29 01:14:44 2011
New Revision: 1140928

URL: http://svn.apache.org/viewvc?rev=1140928&view=rev
Log:
fix CLI parsing of read_repair_chance
patch by Jon Hermes; reviewed by jbellis for CASSANDRA-2837

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

Modified: cassandra/branches/cassandra-0.7/CHANGES.txt
URL: http://svn.apache.org/viewvc/cassandra/branches/cassandra-0.7/CHANGES.txt?rev=1140928&r1=1140927&r2=1140928&view=diff
==============================================================================
--- cassandra/branches/cassandra-0.7/CHANGES.txt (original)
+++ cassandra/branches/cassandra-0.7/CHANGES.txt Wed Jun 29 01:14:44 2011
@@ -25,6 +25,7 @@
  * Fix potential NPE during read repair (CASSANDRA-2823)
  * fix race that could result in Hadoop writer failing to throw an
    exception encountered after close() (CASSANDRA-2755)
+ * fix CLI parsing of read_repair_chance (CASSANDRA-2837)
 
 
 0.7.6

Modified: cassandra/branches/cassandra-0.7/src/java/org/apache/cassandra/cli/CliClient.java
URL: http://svn.apache.org/viewvc/cassandra/branches/cassandra-0.7/src/java/org/apache/cassandra/cli/CliClient.java?rev=1140928&r1=1140927&r2=1140928&view=diff
==============================================================================
--- cassandra/branches/cassandra-0.7/src/java/org/apache/cassandra/cli/CliClient.java (original)
+++ cassandra/branches/cassandra-0.7/src/java/org/apache/cassandra/cli/CliClient.java Wed Jun 29 01:14:44 2011
@@ -902,10 +902,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.");
+                    throw new RuntimeException("Error: read_repair_chance should not be greater than 1.");
 
                 cfDef.setRead_repair_chance(chance);
                 break;