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

Vijay commented on CASSANDRA-3885:
----------------------------------

SliceQueryFilter.serializer needs to deal with Multiple ColumnSlice's or 
at-least we have to create a ticket to followup.

I am confused with the patch a bit... Comments on IndexedSliceReader says
{quote}
forward: [a,b],[d,e],[g,h] reverse: [h,g],[e,d],[b,a].
{quote}

but why do we need to do this?

{code}
            if (reversed && readBackwards)
                currentSlice--;
            else
                currentSlice++;
{code}

In IndexedSliceReader itself is a iterator and within which there is Simple and 
IndexedFetcher which is also kind of a iterator ... it might be better to make 
IndexedSliceReader as abstract class and add functionality into classes. 
                
> Support multiple ranges in SliceQueryFilter
> -------------------------------------------
>
>                 Key: CASSANDRA-3885
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-3885
>             Project: Cassandra
>          Issue Type: Sub-task
>          Components: Core
>            Reporter: Jonathan Ellis
>            Assignee: David Alves
>             Fix For: 1.2
>
>         Attachments: CASSANDRA-3885.patch, CASSANDRA-3885.patch, 
> CASSANDRA-3885.patch, CASSANDRA-3885.patch, CASSANDRA-3885.patch
>
>
> This is logically a subtask of CASSANDRA-2710, but Jira doesn't allow 
> sub-sub-tasks.
> We need to support multiple ranges in a SliceQueryFilter, and we want 
> querying them to be efficient, i.e., one pass through the row to get all of 
> the ranges, rather than one pass per range.
> Supercolumns are irrelevant since the goal is to replace them anyway.  Ignore 
> supercolumn-related code or rip it out, whichever is easier.
> This is ONLY dealing with the storage engine part, not the StorageProxy and 
> Command intra-node messages or the Thrift or CQL client APIs.  Thus, a unit 
> test should be added to ColumnFamilyStoreTest to demonstrate that it works.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to