[GitHub] [flink] danny0405 commented on a change in pull request #12355: [FLINK-17893] [sql-client] SQL CLI should print the root cause if the statement is invalid

2020-06-04 Thread GitBox
danny0405 commented on a change in pull request #12355: URL: https://github.com/apache/flink/pull/12355#discussion_r435085251 ## File path: flink-table/flink-sql-client/src/main/java/org/apache/flink/table/client/cli/CliClient.java ## @@ -253,7 +253,13 @@ public boolean submit

[GitHub] [flink] danny0405 commented on a change in pull request #12355: [FLINK-17893] [sql-client] SQL CLI should print the root cause if the statement is invalid

2020-06-03 Thread GitBox
danny0405 commented on a change in pull request #12355: URL: https://github.com/apache/flink/pull/12355#discussion_r434415772 ## File path: flink-table/flink-sql-client/src/main/java/org/apache/flink/table/client/cli/SqlCommandParser.java ## @@ -253,8 +247,10 @@ private SqlCom

[GitHub] [flink] danny0405 commented on a change in pull request #12355: [FLINK-17893] [sql-client] SQL CLI should print the root cause if the statement is invalid

2020-06-03 Thread GitBox
danny0405 commented on a change in pull request #12355: URL: https://github.com/apache/flink/pull/12355#discussion_r434415304 ## File path: flink-table/flink-sql-client/src/main/java/org/apache/flink/table/client/cli/CliClient.java ## @@ -253,7 +253,13 @@ public boolean submit