You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@calcite.apache.org by "jackylau (Jira)" <ji...@apache.org> on 2023/05/05 12:24:00 UTC

[jira] [Updated] (CALCITE-5674) Cast expr to target type should respect nullable when it is complex type

     [ https://issues.apache.org/jira/browse/CALCITE-5674?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

jackylau updated CALCITE-5674:
------------------------------
    Summary: Cast expr to target type should respect nullable when it is complex type  (was: Cast expr to target type should respect nullable when the type is array)

> Cast expr to target type should respect nullable when it is complex type
> ------------------------------------------------------------------------
>
>                 Key: CALCITE-5674
>                 URL: https://issues.apache.org/jira/browse/CALCITE-5674
>             Project: Calcite
>          Issue Type: Improvement
>    Affects Versions: 1.35.0
>            Reporter: jackylau
>            Priority: Major
>             Fix For: 1.35.0
>
>
> {code:java}
> // code placeholder
>  
> // code placeholder
> sql("select cast(array[1,null,2] as int array) from (values (1))")
> .columnType("INTEGER NOT NULL ARRAY NOT NULL");
> it is not correct, it should return INTEGER ARRAY NOT NULL   {code}
>  
> according the sql standard, cast to type, which can not be nulable and nullable is tabel level attribute which can not changed by cast.
> current the calcite cast will using nullable of expr to replace target type , it is true. but it shoud also set array element type
> {code:java}
> // code placeholder
> /** Derives the type of "CAST(expression AS targetType)". */
> public static RelDataType deriveType(RelDataTypeFactory typeFactory,
>     RelDataType expressionType, RelDataType targetType, boolean safe) {
>   return typeFactory.createTypeWithNullability(targetType,
>       expressionType.isNullable() || safe);
> } {code}
>  



--
This message was sent by Atlassian Jira
(v8.20.10#820010)