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

Thomas D'Silva commented on PHOENIX-4843:
-----------------------------------------

[~jamestaylor]
Thanks for looking at the patch. {{toString()}} was changing the byte[] of  
{{Set<ImmutableBytesPtr> values}} which is used in the evaluate to check if the 
expression is in the list
{code}
if (values.contains(ptr)) {
            ptr.set(PDataType.TRUE_BYTES);
            return true;
        }
{code}
Queries that were filtering on a pk column which was sorted in DESC order were 
not returning any rows (see testInListExpressionWithDesc). I will upload a v2 
patch that keeps the existing behavior of toString() but doesn't modify  
{{values}} set.

> InListExpression toString() converts the values in the list to ASC sort order 
> always
> ------------------------------------------------------------------------------------
>
>                 Key: PHOENIX-4843
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-4843
>             Project: Phoenix
>          Issue Type: Bug
>            Reporter: Thomas D'Silva
>            Assignee: Thomas D'Silva
>            Priority: Major
>             Fix For: 4.15.0, 5.1.0
>
>         Attachments: PHOENIX-4843.patch
>
>




--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to