[FLINK-17728] sql client supports parser statements via sql parser

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.11.0
• Component/s: Table SQL / Client
• Labels: pull-request-available
• Assignee: godfrey he
• Reporter: godfrey he
• Created: 15/May/20 11:48
• Updated: 16/Oct/20 10:50
• Resolved: 19/May/20 10:41

Description

Current, all statements in sql client are parsed via regex matching, which has many limitations, such as it can't handle comments. To avoid that limitations, we should try best to use sql parser to parse a statement. There are many statement can't be handle by sql parser, such as: set, reset. so they are still handle through regex matching.

statements handled through regex matching:
quit, exit, clear, help, desc, explain, set, reset source, show modules

statements handled through sql parser:
show catalogs, show databases, show tables, show functions, use catalog, use, describe, explain plan for, select, insert, DDLs

note: we keep `explain xx`, and also support `explain plan for xx`

Issue Links

• fixes: FLINK-15126 migrate "show functions" from sql cli to sql parser
• relates to: FLINK-17190 SQL client does not support views that reference a table from DDL
• links to: GitHub Pull Request #12188