[GitHub] flink issue #6332: [FLINK-8858] [sql-client] Add support for INSERT INTO in ...

2018-07-14 Thread twalthr
Github user twalthr commented on the issue: https://github.com/apache/flink/pull/6332 Thanks for the review @pnowojski. I agree that we should support `flink-cli < query01.sql` or `echo "INSERT INTO bar SELECT * FROM foo" | flink-cli`. However, I would move this to a separate issue

[GitHub] flink issue #6332: [FLINK-8858] [sql-client] Add support for INSERT INTO in ...

2018-07-13 Thread pnowojski
Github user pnowojski commented on the issue: https://github.com/apache/flink/pull/6332 One more thing. Do we need this `-u` flag? Shouldn't it be enough to support something like: `flink-cli < query01.sql` or `echo "SELECT * FROM foo" | flink-cli` ---