[ 
https://issues.apache.org/jira/browse/CASSANDRA-332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12740887#action_12740887
 ] 

Hudson commented on CASSANDRA-332:
----------------------------------

Integrated in Cassandra #161 (See 
[http://hudson.zones.apache.org/hudson/job/Cassandra/161/])
    final cleanup of SSTableSliceIterator, now with less unnecessary copying of 
Columns around
patch by jbellis; reviewed by Jun Rao for 
avoid copying variables to ColumnGroupReader that it can get from the parent 
class.  rename underscored variables.
patch by jbellis; reviewed by Jun Rao for 
move comparator out of IndexInfo
patch by jbellis; reviewed by Jun Rao for 
always write at least one index (with first and last column of the range) for 
the columns in row.  this vastly simplifies column reading code and makes 
indexing bugs much more obvious (since there is only one read path each for 
names / slices now).
patch by jbellis; reviewed by Jun Rao for 
don't serialize unused column count into column index.  remove DataInput/Output 
round-tripping from ColumnGroupReader
patch by jbellis; reviewed by Jun Rao for 
add test for multi-block reversal.
patch by jbellis; reviewed by Jun Rao for 


> Clean up SSTableSliceIterator to not echo data around DataOutput/Inputs
> -----------------------------------------------------------------------
>
>                 Key: CASSANDRA-332
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-332
>             Project: Cassandra
>          Issue Type: Improvement
>            Reporter: Jonathan Ellis
>            Assignee: Jonathan Ellis
>             Fix For: 0.4
>
>         Attachments: 
> 0001-CASSANDRA-332-add-test-for-multi-block-reversal.txt, 
> 0002-don-t-serialize-unused-column-count-into-column-index.txt, 
> 0003-always-write-indexes-for-at-least-first-and-last-colum.txt, 
> 0004-move-comparator-out-of-IndexInfo.txt, 
> 0005-avoid-copying-variables-to-ColumnGroupReader-that-it-c.txt, 
> 0006-final-cleanup-of-SSTableSliceIterator-now-with-less-u.txt
>
>
> use CFSerializer.deserializeEmpty and then read the columns as necessary, 
> similar to what was done for SSTableNamesIterator

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to