[
https://issues.apache.org/jira/browse/CALCITE-2469?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16583298#comment-16583298
]
pengzhiwei commented on CALCITE-2469:
-------------------------------------
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)
