[jira] [Commented] (CASSANDRA-4180) Remove 2-phase compaction

["Sylvain Lebresne (JIRA)" <ji...@apache.org>]

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

Sylvain Lebresne commented on CASSANDRA-4180:
---------------------------------------------

bq. Removing the row-level bloom filter would make this a lot simpler.

Note that it really matters at the end of the day, but just to make sure we're on the same page, now that the row-level filters have been promoted to the index file, I don't think removing or keeping will have much impact on this ticket. 
                
> Remove 2-phase compaction
> -------------------------
>
>                 Key: CASSANDRA-4180
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-4180
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>            Reporter: Sylvain Lebresne
>            Assignee: Jason Brown
>              Labels: compaction
>             Fix For: 1.3
>
>
> LazilyCompactedRow reads all data twice to compact a row which is obviously inefficient. The main reason we do that is to compute the row header. However, CASSANDRA-2319 have removed the main part of that row header. What remains is the size in bytes and the number of columns, but it should be relatively simple to remove those, which would then remove the need for the two-phase compaction.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira