cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jaakko Laine (JIRA)" <>
Subject [jira] Commented: (CASSANDRA-580) vector clock support
Date Mon, 04 Jan 2010 09:10:54 GMT


Jaakko Laine commented on CASSANDRA-580:

Read through the patch, following comments/questions: 

(1) Utility methods should be put to FBUtilities 

(2) compareByteSubArrays seems to have following issues: (a) it does not handle the case where
bytes2 is null and bytes1 is non-null (b) it does not handle the case where parameter 'length'
is bigger than byte array length 

(3) bytes are copied in manual for loops. Is there a reason for not using arraycopy? It would
perhaps make the code slightly cleaner and faster (don't know about the latter though, as
this involves copying elements within the same array) 

(4) Would it make sense to use some level of data abstraction? Sorting and comparisons always
involve copying & handling of individual bytes, which makes the code slightly cumbersome
to read, and might be more inefficient than using object references as well (again, not sure
about this, have to do some research). 

> vector clock support
> --------------------
>                 Key: CASSANDRA-580
>                 URL:
>             Project: Cassandra
>          Issue Type: New Feature
>          Components: Core
>         Environment: N/A
>            Reporter: Kelvin Kakugawa
>            Assignee: Kelvin Kakugawa
>         Attachments: 580-context-v1.patch, 580-context-v2.patch, 580-context-v3.patch,
580-interface-1-add-vector-clock.diff, 580-interface-2-add-vector-clock.diff, 580-thrift-v3.patch,
580-thrift-v4.patch, 580-thrift-v5.patch, 580-thrift-v6.patch
>   Original Estimate: 672h
>  Remaining Estimate: 672h
> Allow a ColumnFamily to be versioned via vector clocks, instead of long timestamps. 
Purpose: enable incr/decr; flexible conflict resolution.

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

View raw message