lirui-apache commented on issue #9399: [FLINK-13526][sql-client] Switching to a non existing catalog or data… URL: https://github.com/apache/flink/pull/9399#issuecomment-520307510 @bowenli86 Throwing `SqlExecutionException` is part of the contract of `Executor`, so in general I expect SQL client to be able to handle such exceptions. However, since each "call command" is implemented independently (regarding exception handling), I added a test anyway in `CliClientTest` to make sure the exception is indeed handled.
---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services
