[CALCITE-4988] ((A IS NOT NULL OR B) AND A IS NOT NULL) can't be simplify to (A IS NOT NULL) When A is deterministic

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Resolved
• Affects Version/s: 1.29.0
• Fix Version/s: 1.30.0
• Labels: pull-request-available
• Assignee: xiong duan
• Reporter: xiong duan
• Created: 18/Jan/22 02:13
• Updated: 19/Mar/22 09:17
• Resolved: 15/Feb/22 01:45

Description

Now In Calcite:

((A IS NOT NULL OR B) AND A) can be simplify to (A)

When A is a node representing an input reference or field access.

For example:

(name is null or empid = 1) and name is nul -> name is null

But When A is a deterministic node, It can't reduce.

For example:

(replace(name,'e','a') is not null or replace(name,'a','c') is not null) 
and replace(name,'e','a') is not null

Can't be simplify to:

replace(name,'e','a') is not null

This issue originated in CALCITE-4910.

Issue Links

• relates to: CALCITE-4910 Enhance simplify to reduce ((A OR D) AND (A OR C) AND A AND B) to (A AND B)
• links to: GitHub Pull Request #2699