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

Benedict commented on CASSANDRA-6630:
-------------------------------------

If I'm reading this correctly, you're just using ArrayList.add() to shift the 
previously inserted cells around.

This could be severely pathological behaviour for large modification 
statements. Admittedly such large statements should be uncommon, but why not 
add everything in unsorted order and then sort/merge prior to a call to any of 
the accessors?

> Replace UnsortedColumns with ArrayBackedSortedColumns
> -----------------------------------------------------
>
>                 Key: CASSANDRA-6630
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-6630
>             Project: Cassandra
>          Issue Type: Improvement
>            Reporter: Aleksey Yeschenko
>            Assignee: Aleksey Yeschenko
>             Fix For: 2.1
>
>         Attachments: 6630.txt
>
>
> It's possible for a counter mutation to have several CounterUpdateCell-s with 
> the same cell name. Those should be summed up and not ignored.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

Reply via email to