[jira] [Comment Edited] (CASSANDRA-9888) BTreeBackedRow and ComplexColumnData

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

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

Benedict edited comment on CASSANDRA-9888 at 7/24/15 10:20 AM:
---------------------------------------------------------------

Branch available for this [here|https://github.com/belliottsmith/cassandra/tree/9888]

(Waiting on CI results, as this does modify a fairly central piece of functionality. Also, need to introduce some unit tests for the new btree functionality, but that shouldn't hold up review)


was (Author: benedict):
Branch available for this [here|https://github.com/belliottsmith/cassandra/tree/9888]

> BTreeBackedRow and ComplexColumnData
> ------------------------------------
>
>                 Key: CASSANDRA-9888
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-9888
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>            Reporter: Benedict
>            Assignee: Benedict
>             Fix For: 3.0 beta 1
>
>
> I found ArrayBackedRow a little hard to follow, especially around building, so I've converted it to BTreeBackedRow, along with ComplexColumnData. Both now rely on BTree.Builder, which introduces a little extra functionality to permit these classes to be implemented more declaratively. Transformations of these classes are also now uniform and more declarative, also depending on some new functionality in BTree that permits applying a transformation/filtration to an existing btree (this could be optimised at a later date, but should suffice for now).
> The result is IMO both clearer and should scale more gracefully to larger numbers of columns and complex cells.
> This hasn't taken all of the possible improvements of the back of this change to their natural conclusion, as we are somewhat time pressed and I would prefer to get the ball rolling with this first round.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)