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

[jbellis]

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++;
                 }