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

T Jake Luciani commented on CASSANDRA-7279:
-------------------------------------------

bq. what is the rational for not doing anything when we have no comparator?

This is for the case when a replica deserializes the slice over the wire 
internally (from the coordinators request), It's redundant to check in this 
case since the coordinator has already done this check.

bq. Maybe it is redundant checking, but should we explode more violently in 
these cases?

If you run with -ea you get an assertion error.   

bq. I think I found an edge case. Shouldn't the all slice from "" to "" absorb 
"d" to "g" ?

cellname("") get's you another AssertionError.  You should be using 
Composites.EMPTY with that it does the right thing.  These things are checked 
on input

> MultiSliceTest.test_with_overlap* unit tests failing in trunk
> -------------------------------------------------------------
>
>                 Key: CASSANDRA-7279
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-7279
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Tests
>            Reporter: Michael Shuler
>            Assignee: T Jake Luciani
>            Priority: Minor
>             Fix For: 2.1 rc1
>
>         Attachments: 7279-trunk.txt, 7279-trunkv2.txt, 7279-trunkv3.txt
>
>
> Example:
> https://cassci.datastax.com/job/trunk_utest/623/testReport/org.apache.cassandra.thrift/MultiSliceTest/



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to