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

Edward Capriolo commented on CASSANDRA-7279:
--------------------------------------------

Solve the problem in one place. For CQL the code path is skipped in all cases 
currently. When 4762 is added we talking about sorting a list of a few slices 
when the feature is active.  

I like where Jake has it. Put the logic closer to the storage engine, do not 
count multiple components thrift/cql to have to do this separately. As this 
ticket shows it is easy to miss the bug under the assumption that something 
understands some implicit contract.

> 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, 
> 7279-trunkv4.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