[FLINK-17151] Align Calcite's and Flink's SYMBOL types

• Type: Sub-task
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.15.0
• Component/s: Table SQL / API
• Labels: pull-request-available
• Assignee: Timo Walther
• Reporter: Dawid Wysakowicz
• Created: 15/Apr/20 06:33
• Updated: 15/Dec/21 16:48
• Resolved: 15/Dec/21 16:48

Description

Flink's `SymbolType` stores the class of the symbol. Calcite does not. When converting from Flink's to Calcite's type we are loosing that information. We should either remove that from the Flink's type or add it to Calcite's.

I am in favor of removing it from the SymbolType. This type is a pseudo type which can be used only for literals. The symbol class is already part of the corresponding value, therefore it is a redundant information in the type itself.

Initial discussion: https://github.com/apache/flink/pull/11694#discussion_r408058640

Issue Links

• links to: GitHub Pull Request #18107