You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@calcite.apache.org by "Alessandro Solimando (Jira)" <ji...@apache.org> on 2023/04/25 14:32:00 UTC

[jira] [Resolved] (CALCITE-5639) RexSimplify should remove IS NOT NULL check when LIKE comparison is present

     [ https://issues.apache.org/jira/browse/CALCITE-5639?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Alessandro Solimando resolved CALCITE-5639.
-------------------------------------------
    Fix Version/s: 1.35.0
       Resolution: Fixed

Fixed via [0305067|https://github.com/apache/calcite/commit/03050674594152ea785af69517f33960d7e27dd5], thanks [~lchistov1987] for the patch and [~julianhyde] [~Chunwei Lei] and [~libenchao] for your comments and reivews!

> RexSimplify should remove IS NOT NULL check when LIKE comparison is present
> ---------------------------------------------------------------------------
>
>                 Key: CALCITE-5639
>                 URL: https://issues.apache.org/jira/browse/CALCITE-5639
>             Project: Calcite
>          Issue Type: Bug
>    Affects Versions: 1.34.0
>            Reporter: Leonid Chistov
>            Assignee: Leonid Chistov
>            Priority: Minor
>              Labels: pull-request-available
>             Fix For: 1.35.0
>
>          Time Spent: 3.5h
>  Remaining Estimate: 0h
>
> Consider query like
> {code:java}
> select r_reason_desc from reason
> where r_reason_desc is not null and r_reason_desc like '%child%' 
> {code}
> where "r_reason_desc" is a nullable field.
> When RexSimplify::simplifyFilterPredicates is called on that conjunction of expressions, expression is not simplified, meaning that redundant  "is not null" check is not removed.
> In the same time, if query like 
> {code:java}
> select r_reason_desc from reason
> where r_reason_desc is not null and r_reason_desc >= 'A'{code}
> is passed to optimizer, redundant "is not null" check is eliminated.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)