fsk119 commented on a change in pull request #15366:
URL: https://github.com/apache/flink/pull/15366#discussion_r602851752
##########
File path:
flink-table/flink-sql-client/src/test/java/org/apache/flink/table/client/cli/CliClientTest.java
##########
@@ -129,6 +142,126 @@ public void testHistoryFile() throws Exception {
}
}
+ @Test
+ public void testSqlFileContainsUnknownStatement() throws Exception {
Review comment:
We have support the option `sql-client.verbose` now. Users can print the
full stack now.
Sql client just report the exception.. It's the Calcite parser that doesn't
give us meaningful exception message.
Maybe we can print `Could not parse SQL statement` if the parse fails.
Otherwise print `Could not execute SQL statement`.
--
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]
