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

Kelvin Kakugawa commented on CASSANDRA-1072:
--------------------------------------------

So, the challenge is that I mirrored the mutation interface for counters.

I have two proposals that I'd like your opinion on:
a) add an optional uuid field to CounterColumn, or
b) add an optional uuid field to CounterMutation that covers the whole mutation.

(b) is not very fine-grained.  However, if a batch_add call fails, the client 
should retry the whole batch mutation.  ntm, uuids are specific to a given 
counter, so there won't be any fear of collisions across keys.  Although, if a 
batch mutation were to update the same key twice (w/in its update_map), the 
mutation will have to be collapsed at some point.

note: if we go w/ (a), then CounterUpdate should be refactored out into just a 
CounterColumn.

> Increment counters
> ------------------
>
>                 Key: CASSANDRA-1072
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-1072
>             Project: Cassandra
>          Issue Type: Sub-task
>          Components: Core
>            Reporter: Johan Oskarsson
>            Assignee: Kelvin Kakugawa
>         Attachments: CASSANDRA-1072.120610.patch, CASSANDRA-1072.patch, 
> increment_test.py, Partitionedcountersdesigndoc.pdf
>
>
> Break out the increment counters out of CASSANDRA-580. Classes are shared 
> between the two features but without the plain version vector code the 
> changeset becomes smaller and more manageable.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to