[
https://issues.apache.org/jira/browse/CALCITE-4039?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17124698#comment-17124698
]
TANG Wen-hui commented on CALCITE-4039:
---------------------------------------
Calcite validate phase does not miss LATERAL operator, since it has already set
the parameter lateral of registerFrom to be true When encountering the LATERAL
operator, see
https://github.com/apache/calcite/blob/29f798fb6919f24d95776658b7e659af179a8b15/core/src/main/java/org/apache/calcite/sql/validate/SqlValidatorImpl.java#L2178.
And when converted to RelNode, it works.
If you want to convert to sql string, you'd better use RelToSqlConverter,
RelToSqlConverter#visit can help. see
https://github.com/apache/calcite/blob/29f798fb6919f24d95776658b7e659af179a8b15/core/src/main/java/org/apache/calcite/rel/rel2sql/RelToSqlConverter.java#L279
> Missing LATERAL keyword after sql validate
> ------------------------------------------
>
> Key: CALCITE-4039
> URL: https://issues.apache.org/jira/browse/CALCITE-4039
> Project: Calcite
> Issue Type: Bug
> Components: core
> Affects Versions: 1.23.0
> Reporter: YufeiLiu
> Priority: Major
>
> I tried format to sql string after validate SqlNode, but the {{LATERAL}}
> keyword was missing in the result.
> original sql:
> {code:sql}
> SELECT `B`, `C`, `D`, `E`
> FROM `source_table`,
> LATERAL TABLE(`MY_TABLE_FUNCTION`(`A`)) AS `T` (`B`, `C`, `D`, `E`)
> {code}
> validated sql:
> {code:sql}
> SELECT `B`, `C`, `D`, `E`
> FROM `source_table`,
> TABLE(`MY_TABLE_FUNCTION`(`A`)) AS `T` (`B`, `C`, `D`, `E`)
> {code}
> In {{SqlValidatorImpl}}, only return newOperand and missing the LATERAL
> operator.
> {code:java}
> case LATERAL:
> return registerFrom(
> parentScope,
> usingScope,
> register,
> ((SqlCall) node).operand(0),
> enclosingNode,
> alias,
> extendList,
> forceNullable,
> true);
> {code}
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
