[jira] Commented: (CASSANDRA-332) Clean up SSTableSliceIterator to not echo data around DataOutput/Inputs

["Jonathan Ellis (JIRA)" <ji...@apache.org>]

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

Jonathan Ellis commented on CASSANDRA-332:
------------------------------------------

06
    final cleanup of SSTableSliceIterator, now with less unnecessary copying of Columns around

05
    avoid copying variables to ColumnGroupReader that it can get from the parent class.  rename underscored variables

04
    move comparator out of IndexInfo

03
    always write indexes for at least first and last 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).

02
    don't serialize unused column count into column index.  remove DataInput/Output round-tripping from ColumnGroupRe

01
    CASSANDRA-332 add test for multi-block reversal


> 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: Sub-task
>            Reporter: Jonathan Ellis
>         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.