[
https://issues.apache.org/jira/browse/FLINK-31464?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17702546#comment-17702546
]
Timo Walther commented on FLINK-31464:
--------------------------------------
However, the question arises why we need resolved identifiers at this location
or why alter table needs resolved tables at this location. In the end, the POJO
structure should represent an AST without any bigger validation. Validation can
be done by the operation execution logic.
> Move SqlNode conversion logic out from SqlToOperationConverter
> --------------------------------------------------------------
>
> Key: FLINK-31464
> URL: https://issues.apache.org/jira/browse/FLINK-31464
> Project: Flink
> Issue Type: Technical Debt
> Components: Table SQL / Planner
> Reporter: luoyuxia
> Assignee: xuzhiwen
> Priority: Major
>
> Similar to FLINK-31368, the `SqlToOperationConverter` is a bit bloated. We
> can refactor it to avoid the code length for this class grow quickly.
> We can follow the idea proposed in FLINK-31368.
>
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
