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

Sylvain Lebresne commented on CASSANDRA-6638:
---------------------------------------------

bq. And then a loop is started over all the ifile entries until end of ifile 
file (there is no upper boundary check)!

There is an upper check, we compare each deserialized index key to the range 
we're "seeking to the start" of. If we've seek to a key in the third range, 
that index key will sort after the 2nd range start and we'll exist the loop 
right away.

> SSTableScanner can Skip Rows with vnodes
> ----------------------------------------
>
>                 Key: CASSANDRA-6638
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-6638
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Core
>            Reporter: Tyler Hobbs
>            Assignee: Sylvain Lebresne
>            Priority: Blocker
>             Fix For: 2.0.5
>
>         Attachments: 6638-repro-test.txt, 6638.txt
>
>
> CASSANDRA-2524 added multiple range support to SSTableScanner, but it looks 
> like there is at least one case where keys can be skipped.  This can result 
> in cleanup removing legitimate keys.
> See the attached patch that adds a unit test to reproduce the case.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

Reply via email to