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

Tyler Hobbs commented on CASSANDRA-6950:
----------------------------------------

bq. I don't feel extremely strongly though, if we prefer going with changing 
thrift too, I'll just blame it on you if some user complains that we've break 
his code in a minor release :).

Hah, okay, I'll gladly take the blame if this breaks somebody's very strange 
code :).  Fortunately it would be very easy to fix in that case: remove the 
ReversedType validator or reverse the query operator.

bq. In thrift we never (to the best of my knowledge) special case code for a 
specific AbstractType, and starting doing so now in that case makes me slightly 
uncomfortable.

I understand that feeling, but we probably should have just disallowed 
ReversedType for column validators in the first place.

{{ExtendedFilter.isSatisfiedBy()}} seems like the more logical and future-proof 
place to make the fix, so I vote for that.

> Secondary index query fails with tc range query when ordered by DESC
> --------------------------------------------------------------------
>
>                 Key: CASSANDRA-6950
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-6950
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Core
>         Environment: RHEL 6.3 virtual guest, 
> apache-cassandra-2.0.6-SNAPSHOT-src.tar.gz from build #284 (also tried with 
> 2.0.5 with CASSANDRA-6666 patch custom-applied with same result).
>            Reporter: Andre Campeau
>            Assignee: Sylvain Lebresne
>             Fix For: 2.0.8
>
>         Attachments: 6950-pycassa-repro.py, 6950.txt
>
>
>     create table test4 ( name text, lname text, tc bigint, record text, 
> PRIMARY KEY ((name, lname), tc)) WITH CLUSTERING ORDER BY (tc DESC) AND 
> compaction={'class': 'LeveledCompactionStrategy'};
>     create index test4_index ON test4(lname);
> Populate it with some data and non-zero tc values, then try:
>     select * from test4 where lname='blah' and tc>0 allow filtering;
> And, (0 rows) returned, even though there are rows which should be found.
> When I create the table using CLUSTERING ORDER BY (tc ASC), the above query 
> works. Rows are correctly returned based on the range check.
> Tried various combinations but with descending order on tc nothing works.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to