[jira] [Commented] (CASSANDRA-5712) Reverse slice queries can skip range tombstones

["Jonathan Ellis (JIRA)" <ji...@apache.org>]

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

Jonathan Ellis commented on CASSANDRA-5712:
-------------------------------------------

Do we really need rangeTombstonesReversed?  ISTM that we could just special case tombstones in addColumn (rename to addAtom) to always addFirst regardless of reversed-ness.
                
> Reverse slice queries can skip range tombstones
> -----------------------------------------------
>
>                 Key: CASSANDRA-5712
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-5712
>             Project: Cassandra
>          Issue Type: Bug
>    Affects Versions: 1.2.0
>            Reporter: Sylvain Lebresne
>            Assignee: Sylvain Lebresne
>             Fix For: 1.2.7
>
>         Attachments: 5712.txt
>
>
> On disk, we represent range tombstones by a marker at the beginning of the range covered. Since we repeat such markers when they overlap an index block and since an index block is always read in forward order (even in reverse queries), we are guaranteed to see a range tombstone before any column it covers. However, IndexedSliceReader returns the columns of an index block in reverse order and thus can return a range tombstone *after* columns it covers.
> It follows that some range tombstone can be skipped during a reversed range slice. We need to fix IndexedSliceReader to always return range tombstone first (or at least before the first column covered by each range tombstone).

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira