[FLINK-34312] Improve the handling of default node types for named parameters.

• Type: Sub-task
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.20.0
• Labels: pull-request-available
• Assignee: Feng Jin
• Reporter: Feng Jin
• Created: 30/Jan/24 16:43
• Updated: 26/Feb/24 11:28
• Resolved: 26/Feb/24 11:28

Description

Currently, we have supported the use of named parameters with optional arguments.

By adapting the interface of Calcite, we can fill in the default operator when a parameter is missing. Whether it is during the validation phase or when converting to SqlToRel phase, we need to handle it specially by modifying the return type of DEFAULT operator based on the argument type of the operator.
We have multiple places that need to handle the type of DEFAULT operator, including SqlCallBinding, SqlOperatorBinding, and SqlToRelConverter.

The improved solution is as follows:

Before SqlToRel, we can construct a DEFAULT node with a return type that matches the argument type. This way, during the SqlToRel phase, there is no need for special handling of the DEFAULT node's type.

Issue Links

• links to: GitHub Pull Request #24235