[FLINK-22770] Expose SET/RESET from the parser

• Type: New Feature
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.13.0
• Fix Version/s: 1.14.0, 1.13.2
• Component/s: Table SQL / API, Table SQL / Planner
• Labels: pull-request-available
• Assignee: Ingo Bürk
• Reporter: Ingo Bürk
• Created: 25/May/21 10:58
• Updated: 30/Sep/21 14:26
• Resolved: 28/May/21 10:41

Description

The SET/RESET statements are currently implemented using regular expressions in the ExtendedParser. They should instead be properly added to the grammar and parser.

In a second step, the ExtendedParser implementation for it can be removed. However, this would be a breaking change as key/value would then have to be quoted (as they are in other places, i.e. table options) and no longer be treated as SQL identifiers

Issue Links

• relates to: FLINK-22848 Deprecate unquoted options for SET / RESET
• relates to: FLINK-23852 In sql-client mode ，could not change the properties.
• relates to: FLINK-22847 SET should print options quoted
• relates to: FLINK-23934 table doc timezone page 'table.local-time-zone' Configuration error
• links to: GitHub Pull Request #16006
• links to: GitHub Pull Request #17141