[GitHub] [flink] wuchong commented on a change in pull request #15458: [FLINK-22064][table] Don't submit statement set when no insert is added in the sql client

[GitBox]

wuchong commented on a change in pull request #15458:
URL: https://github.com/apache/flink/pull/15458#discussion_r605363528



##########
File path: 
flink-table/flink-sql-client/src/main/java/org/apache/flink/table/client/cli/CliStrings.java
##########
@@ -212,6 +212,9 @@ private CliStrings() {
 
     public static final String MESSAGE_BEGIN_STATEMENT_SET = "Begin a 
statement set.";
 
+    public static final String MESSAGE_NO_INSERT_STATEMENT_IN_STATEMENT_SET =
+            "No insert statement in statement set, skip submit.";

Review comment:
       ```suggestion
               "No statement in the statement set, skip submit.";
   ```
   
   I think we don't need to mention the statement kind, we may support more DML 
statements, e.g. `MERGE INTO` in the future. 




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