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

Hudson commented on CASSANDRA-2641:
-----------------------------------

Integrated in Cassandra-0.8 #122 (See 
[https://builds.apache.org/hudson/job/Cassandra-0.8/122/])
    Assert ranges are not overlapping in AbstractBounds.normalize
patch by stuhood; reviewed by slebresne for CASSANDRA-2641

slebresne : 
http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1126389
Files : 
* /cassandra/branches/cassandra-0.8/CHANGES.txt
* 
/cassandra/branches/cassandra-0.8/test/unit/org/apache/cassandra/streaming/StreamingTransferTest.java
* 
/cassandra/branches/cassandra-0.8/src/java/org/apache/cassandra/dht/AbstractBounds.java


> AbstractBounds.normalize should deal with overlapping ranges
> ------------------------------------------------------------
>
>                 Key: CASSANDRA-2641
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-2641
>             Project: Cassandra
>          Issue Type: Test
>          Components: Core
>            Reporter: Stu Hood
>            Assignee: Stu Hood
>            Priority: Minor
>             Fix For: 0.8.1
>
>         Attachments: 0001-Assert-non-overlapping-ranges-in-normalize.txt, 
> 0002-Don-t-use-overlapping-ranges-in-tests.txt
>
>
> Apparently no consumers have encountered it in production, but 
> AbstractBounds.normalize does not handle overlapping ranges. If given 
> overlapping ranges, the output will be sorted but still overlapping, for 
> which SSTableReader.getPositionsForRanges will choose ranges in an SSTable 
> that may overlap.
> We should either add an assert in normalize(), or in getPositionsForRanges() 
> to ensure that this never bites us in production.

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

Reply via email to