cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Benedict (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (CASSANDRA-6630) Replace UnsortedColumns with ArrayBackedSortedColumns
Date Thu, 30 Jan 2014 15:32:13 GMT

    [ 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)

Mime
View raw message