[
https://issues.apache.org/jira/browse/CALCITE-6993?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17948383#comment-17948383
]
Stamatis Zampetakis commented on CALCITE-6993:
----------------------------------------------
For the nullable information in the Array I am not sure if it is "necessary".
{noformat}
2: It is necessary to add the nullable information output for the
Array.{noformat}
There are certain rules inside the RexLiteral that define when nullability
should appear in the digest and when not. We have to review those carefully.
> Simplify Sarg digest to drop redundant type information
> -------------------------------------------------------
>
> Key: CALCITE-6993
> URL: https://issues.apache.org/jira/browse/CALCITE-6993
> Project: Calcite
> Issue Type: Bug
> Reporter: xiong duan
> Priority: Major
>
> The issue was discovered at [https://github.com/apache/calcite/pull/4345.]
> For example in unit test testMakeInReturnsSearchForArrayLiterals:
> {code:java}
> [100:INTEGER, 200:INTEGER]:INTEGER NOT NULL ARRAY,[300:INTEGER,
> 400:INTEGER]:INTEGER NOT NULL ARRAY]:INTEGER NOT NULL ARRAY
> Can be simplified to：
> [100, 200],[300, 400]:INTEGER NOT NULL ARRAY NOT NULL{code}
> 1: The repeated element types do not necessarily need to be output.
> 2: It is necessary to add the nullable information output for the Array.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
