You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@phoenix.apache.org by "Toshihiro Suzuki (Jira)" <ji...@apache.org> on 2020/08/24 08:14:00 UTC

[jira] [Comment Edited] (PHOENIX-6098) IndexPredicateAnalyzer wrongly handles pushdown predicates and residual predicates

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

Toshihiro Suzuki edited comment on PHOENIX-6098 at 8/24/20, 8:13 AM:
---------------------------------------------------------------------

I created a PR for this. It's very hard to add tests to phoenix-hive module of phoenix-connectors, so I didn't add any tests.


was (Author: brfrn169):
I created a PR for this. It's very hard to add tests to phoenixp-hive module of phoenix-connectors, so I didn't add any tests.

> IndexPredicateAnalyzer wrongly handles pushdown predicates and residual predicates
> ----------------------------------------------------------------------------------
>
>                 Key: PHOENIX-6098
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-6098
>             Project: Phoenix
>          Issue Type: Bug
>          Components: hive-connector
>            Reporter: Toshihiro Suzuki
>            Assignee: Toshihiro Suzuki
>            Priority: Major
>
> Currently, the following code of IndexPredicateAnalyzer is assuming that GenericUDFOPAnd always has 2 children nodes. I think this is wrong and it leads wrong results:
> https://github.com/apache/phoenix-connectors/blob/5bd23ae2a0f70c3b3edf92a53780dafa643faf26/phoenix-hive/src/main/java/org/apache/phoenix/hive/ql/index/IndexPredicateAnalyzer.java#L346-L359



--
This message was sent by Atlassian Jira
(v8.3.4#803005)