cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Yuki Morishita (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (CASSANDRA-4308) Promote the use of IFilter for internal commands
Date Thu, 07 Jun 2012 19:49:24 GMT

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

Yuki Morishita commented on CASSANDRA-4308:
-------------------------------------------

LGTM.

nit: You can now replace QueryFilter.getFilter(SlicePredicate, AbstractType) which is only
used in deprecated IndexScanCommand and ColumnFamilyStoreTest,  with newly added ThriftValidation.asIFilter.
                
> Promote the use of IFilter for internal commands 
> -------------------------------------------------
>
>                 Key: CASSANDRA-4308
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-4308
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>            Reporter: Sylvain Lebresne
>            Assignee: Sylvain Lebresne
>            Priority: Minor
>             Fix For: 1.2
>
>         Attachments: 4308.txt
>
>
> All read commands (the IReadCommand) work on slice and names filters, but none of them
uses the SliceQueryFilter and NamesQueryFilter classes (RangeSliceCommand uses SlicePrediate
from thrift and {SliceFrom, SliceByNames}ReadReadCommand interns the arguments).
> The main problem of that is that it follows that those command don't share the serialization
code for the column filters. Which isn't good for code reuse, but also makes it a pain to
add new fields to the filter SliceQueryFilter (which CASSANDRA-3885 will need, but probably
CASSANDRA-3647 too).

--
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

        

Mime
View raw message