[CALCITE-4325] RexSimplify/Sarg incorrectly simplifies complex expressions with NULL

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.26.0
• Fix Version/s: 1.27.0
• Component/s: core
• Labels: pull-request-available

Environment

All environments

• Assignee: Julian Hyde
• Reporter: Igor Lozynskyi
• Created: 10/Oct/20 09:52
• Updated: 03/Jun/21 22:33
• Resolved: 15/Oct/20 21:50

Description

In Calcite 1.26, the Rex expressions like follows:

(deptno = 20 OR deptno IS NULL) AND deptno = 10

Are simplified to:

deptno IS NULL

Instead of:

FALSE

Similarly, the following expression:

(deptno <> 20 OR deptno IS NULL) AND deptno = 10

Is simplified to:

deptno = 10 OR deptno IS NULL

Instead of:

deptno = 10

A diff with related unit tests (for RelBuilderTest.java) is in the attachment.

Issue Links

• is related to: CALCITE-4173 Add internal SEARCH operator and Sarg literal, replacing use of IN in RexCall
• links to: GitHub Pull Request #2212