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

Ariel Weisberg commented on CASSANDRA-8730:
-------------------------------------------

I've gone over this a couple of times. It looks okay and the tests are 
persuasive that they check that the comparisons are self-consistent. Maybe 
resurrect the old versions to check that they are prior version consistent in 
the unit tests. Maybe modify the test to use a seeded RNG with a known seed and 
print it out so if there is a failure we can reproduce.

UUIDType is supposed to correctly compare TimeUUIDTypes and there is some cut 
and paste so it maybe it should use TimeUUID in parallel and see if both 
approaches return the same answer when LHS and RHS are both Time UUIDs?

What is UUIDTypeTest.permute used to check for?

> Optimize UUIDType comparisons
> -----------------------------
>
>                 Key: CASSANDRA-8730
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-8730
>             Project: Cassandra
>          Issue Type: Improvement
>            Reporter: J.B. Langston
>            Assignee: Benedict
>             Fix For: 3.0
>
>
> Compaction is slow on tables using compound keys containing UUIDs due to 
> being CPU bound by key comparison.  [~benedict] said he sees some easy 
> optimizations that could be made for UUID comparison.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to