You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@calcite.apache.org by "pengzhiwei (JIRA)" <ji...@apache.org> on 2018/08/17 06:06:00 UTC

[jira] [Issue Comment Deleted] (CALCITE-2469) RexSimplify should not optimize '(not x) is null' to 'x is not null

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

pengzhiwei updated CALCITE-2469:
--------------------------------
    Comment: was deleted

(was: Hi @[~vladimirsitnikov] 
{noformat}
|   I'm not convinced why another method is needed.
|   AFAIK, all you need to do regarding this bug is to add proper cases to negateNullSafe{noformat}
Currently,RexSimplify has not simplified IS NULL expression but done it for IS NOT NULL in  simplifyIsNotNull,so 'not(x) is null' has not simplified to 'x is null'. If we want to do this simplification for IS NULL, a simplifyIsNull method should be added just like the simplifyIsNotNull do. Maybe I can do this in another issue,and just fix this bug here.)

> RexSimplify should not optimize '(not x) is null' to 'x is not null
> -------------------------------------------------------------------
>
>                 Key: CALCITE-2469
>                 URL: https://issues.apache.org/jira/browse/CALCITE-2469
>             Project: Calcite
>          Issue Type: Bug
>          Components: core
>    Affects Versions: 1.17.0
>            Reporter: pengzhiwei
>            Assignee: Julian Hyde
>            Priority: Critical
>
> The RexSimplify will optimize the '(not x) is null' to 'x is not null'. However,this is incorrect.It is same to the case '(not x) is not null' .
>  



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