calcite
calcite copied to clipboard
[CALCITE-4725] Between clause operands checker should check all combi…
…nations of the three operands
What is the purpose of the change
Fix the issue that between clause operands checker should check all combinations of the three operands, as what has been discuss in CALCITE-4725 in ASF Jira.
Brief change log
- core/src/main/java/org/apache/calcite/sql/type/SameOperandTypeChecker.java
- core/src/main/java/org/apache/calcite/sql/type/SameOperandTypeExceptLastOperandChecker.java
@paul8263 please add unit test in this patch.
Hi @NobiGo , Thank you for your advice. Unit test has been added and several errors have been fixed.
Hi @NobiGo , Could you please help review this issue?
I plan to review this PR. It looks like BETWEEN cannot currently compare TIMESTAMPS, for example