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

Xintong Song updated FLINK-19362:
---------------------------------
    Fix Version/s: 1.18.0
                       (was: 1.17.0)

> Remove confusing comment for `DOT` operator codegen
> ---------------------------------------------------
>
>                 Key: FLINK-19362
>                 URL: https://issues.apache.org/jira/browse/FLINK-19362
>             Project: Flink
>          Issue Type: Improvement
>          Components: Table SQL / Runtime
>    Affects Versions: 1.11.0
>            Reporter: hailong wang
>            Priority: Minor
>              Labels: auto-deprioritized-major
>             Fix For: 1.18.0
>
>
> `DOT` operator codegen (ExprCodeGenerator#generateCallExpression) has comment 
> as following:
> {code:java}
> // due to https://issues.apache.org/jira/browse/CALCITE-2162, expression such 
> as
> // "array[1].a.b" won't work now.
> if (operands.size > 2) {
>   throw new CodeGenException(
>     "A DOT operator with more than 2 operands is not supported yet.")
> }
> {code}
> But `array[1].a.b` actually can work for flink job. `DOT` will be transform 
> to `RexFieldAccess` for CALCITE-2542. And `generateDot` will never be invoked 
>  except suppporting ITEM for ROW types.
> Simply, I think we can only delete the comment which is confusing. 



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

Reply via email to