Github user twalthr commented on a diff in the pull request:
https://github.com/apache/flink/pull/6332#discussion_r202507463
--- Diff: flink-libraries/flink-sql-client/src/main/java/org/apache/flink/table/client/SqlClient.java
---
@@ -97,14 +97,34 @@ private void start() {
// add shutdown hook
Runtime.getRuntime().addShutdownHook(new EmbeddedShutdownThread(context, executor));
- // start CLI
- final CliClient cli = new CliClient(context, executor);
- cli.open();
+ // do the actual work
+ openCli(context, executor);
} else {
throw new SqlClientException("Gateway mode is not supported yet.");
}
}
+ /**
+ * Opens the CLI client for executing SQL statements.
+ *
+ * @param context session context
+ * @param executor executor
+ */
+ private void openCli(SessionContext context, Executor executor) {
+ final CliClient cli = new CliClient(context, executor);
+ // interactive CLI mode
+ if (options.getUpdateStatement() == null) {
+ cli.open();
+ }
+ // execute single update statement
+ else {
+ final boolean success = cli.submitUpdate(options.getUpdateStatement());
--- End diff --
No, this would block the process for unbounded queries and require a (fault-tolerant)
monitoring in the SQL Client which is not intended. We just block until the statement has
been submitted to the cluster.
---
|