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;