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

Jonathan Ellis commented on CASSANDRA-2233:
-------------------------------------------

bq. I'm not sure how useful the natural order of the other UUID versions is

Agreed.

bq. don't access ByteBuffer.array() directly without asking if 
ByteBuffer.hasArray() first

Yes, we have a bunch of methods in ByteBufferUtil that will use array if it 
exists otherwise will either duplicate and use relative methods or use 
byte-at-a-time get.

bq. write a unit test.

+1, mostly it will just need to adapt what we have in TimeUUIDTypeTeset

> Add unified UUIDType
> --------------------
>
>                 Key: CASSANDRA-2233
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-2233
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>    Affects Versions: 0.7.3
>            Reporter: Ed Anuff
>            Priority: Minor
>         Attachments: UUIDType.java
>
>
> Unified UUIDType comparator, compares as time-based if both UUIDs are 
> time-based, otherwise uses byte comparison.  Based on code from the current 
> LexicalUUIDType and TimeUUIDType comparers, so performance and behavior 
> should be consistent and compatible.

-- 
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to