xuefuz commented on issue #8738: [FLINK-12845][sql-client] Execute multiple 
statements in command line…
URL: https://github.com/apache/flink/pull/8738#issuecomment-524948104
 
 
   I agree the current PR is way too hacky. However, I don't see the need  of 
extending SQL parser just to support the multi-command line either as there can 
be commands that's not SQL at all. As far as I can see, we need to split the 
command line by ';' (certainly take care of possible escaped ';'), pre-process 
each segment to determine the command type (which we are already doing), and 
invoke sql parser only if the command is a sql command.

----------------------------------------------------------------
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