You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by sl...@apache.org on 2011/11/15 08:39:12 UTC

svn commit: r1202080 - in /cassandra/branches/cassandra-1.0: CHANGES.txt src/java/org/apache/cassandra/io/compress/CompressionParameters.java

Author: slebresne
Date: Tue Nov 15 07:39:12 2011
New Revision: 1202080

URL: http://svn.apache.org/viewvc?rev=1202080&view=rev
Log:
Fix bad handling of the chunk_length_kb compression option
patch by slebresne; reviewed by xedin for CASSANDRA-3492

Modified:
    cassandra/branches/cassandra-1.0/CHANGES.txt
    cassandra/branches/cassandra-1.0/src/java/org/apache/cassandra/io/compress/CompressionParameters.java

Modified: cassandra/branches/cassandra-1.0/CHANGES.txt
URL: http://svn.apache.org/viewvc/cassandra/branches/cassandra-1.0/CHANGES.txt?rev=1202080&r1=1202079&r2=1202080&view=diff
==============================================================================
--- cassandra/branches/cassandra-1.0/CHANGES.txt (original)
+++ cassandra/branches/cassandra-1.0/CHANGES.txt Tue Nov 15 07:39:12 2011
@@ -14,6 +14,7 @@
  * Fix race introduced by CASSANDRA-2503 (CASSANDRA-3482)
  * Fix incomplete deletion of delivered hints (CASSANDRA-3466)
  * Avoid reschuling compactions when no compaction was executed (CASSANDRA-3483)
+ * fix handling of the chunk_length_kb compression options (CASSANDRA-3492)
 Merged from 0.8:
  * Make counter shard merging thread safe (CASSANDRA-3178)
  * fix updating CF row_cache_provider (CASSANDRA-3414)

Modified: cassandra/branches/cassandra-1.0/src/java/org/apache/cassandra/io/compress/CompressionParameters.java
URL: http://svn.apache.org/viewvc/cassandra/branches/cassandra-1.0/src/java/org/apache/cassandra/io/compress/CompressionParameters.java?rev=1202080&r1=1202079&r2=1202080&view=diff
==============================================================================
--- cassandra/branches/cassandra-1.0/src/java/org/apache/cassandra/io/compress/CompressionParameters.java (original)
+++ cassandra/branches/cassandra-1.0/src/java/org/apache/cassandra/io/compress/CompressionParameters.java Tue Nov 15 07:39:12 2011
@@ -137,6 +137,9 @@ public class CompressionParameters
         return compressionOptions;
     }
 
+    /**
+     * Parse the chunk length (in KB) and returns it as bytes.
+     */
     private static Integer parseChunkLength(String chLength) throws ConfigurationException
     {
         if (chLength == null)
@@ -144,7 +147,7 @@ public class CompressionParameters
 
         try
         {
-            return Integer.parseInt(chLength);
+            return 1024 * Integer.parseInt(chLength);
         }
         catch (NumberFormatException e)
         {