bowenli86 edited a comment 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-504704488
 
 
   @KurtYoung Extremely simple commands like this can be on sql cli itself IMO. 
DDL/DML is better to go thru sql parser.
   
   To be clear, I'm not against migrating these simple commands to sql parser 
*later* if we want, though I don't think it's that necessary given the 
complexity. Right now it's about making e2e user experience complete for 
catalogs in sql cli 1.9, and sql parser is too uncertain at this point for me 
to rely on.

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to