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 2010/10/19 01:04:28 UTC

svn commit: r1024058 - /cassandra/trunk/src/java/org/apache/cassandra/io/sstable/SSTableWriter.java

Author: jbellis
Date: Mon Oct 18 23:04:28 2010
New Revision: 1024058

URL: http://svn.apache.org/viewvc?rev=1024058&view=rev
Log:
compute next row position before changing the file pointer.  patch by jbellis for CASSANDRA-1620

Modified:
    cassandra/trunk/src/java/org/apache/cassandra/io/sstable/SSTableWriter.java

Modified: cassandra/trunk/src/java/org/apache/cassandra/io/sstable/SSTableWriter.java
URL: http://svn.apache.org/viewvc/cassandra/trunk/src/java/org/apache/cassandra/io/sstable/SSTableWriter.java?rev=1024058&r1=1024057&r2=1024058&view=diff
==============================================================================
--- cassandra/trunk/src/java/org/apache/cassandra/io/sstable/SSTableWriter.java (original)
+++ cassandra/trunk/src/java/org/apache/cassandra/io/sstable/SSTableWriter.java Mon Oct 18 23:04:28 2010
@@ -286,13 +286,14 @@ public class SSTableWriter extends SSTab
                     iwriter.afterAppend(key, rowPosition);
 
                     long dataSize = SSTableReader.readRowSize(dfile, desc);
+                    rowPosition = dfile.getFilePointer() + dataSize; // next row
+
                     IndexHelper.skipBloomFilter(dfile);
                     IndexHelper.skipIndex(dfile);
                     ColumnFamily.serializer().deserializeFromSSTableNoColumns(ColumnFamily.create(cfs.metadata), dfile);
                     rowSizes.add(dataSize);
                     columnCounts.add(dfile.readInt());
 
-                    rowPosition = dfile.getFilePointer() + dataSize;
                     dfile.seek(rowPosition);
                     rows++;
                 }