[GitHub] [flink] danny0405 commented on issue #8830: [FLINK-12933][sql client] support 'use catalog' and 'use database' in SQL CLI

[GitBox <gi...@apache.org>]

danny0405 commented on issue #8830: [FLINK-12933][sql client] support 'use catalog' and 'use database' in SQL CLI
URL: https://github.com/apache/flink/pull/8830#issuecomment-504803156
 
 
   @KurtYoung @bowenli86 Since we have a new module named `flink-sql-parser` aimed to handle all the sql parse work including DDL/DML/DQL, which means the parser module is the only entrance for any sql parse work in the future, the `Use catalog` and `Use database` are also expected to put there.
   
   IMHO, sql client could have its own commands but should exclude the DDL/DML/DQL,  which is the duty of sql parser.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services