dawidwys commented on issue #9781: [FLINK-14098] Support multiple statements for TableEnvironment. URL: https://github.com/apache/flink/pull/9781#issuecomment-536464957 Hi @wuchong, personally I think changing `List<Operation>` to `Iterable<Operation>` makes sense. I do have though bigger concerns regarding the lazy execution of queries and I wonder if we should not revisit that as well. The problem I see what happens if we have a sequence of statements like this: ``` CREATE TABLE test (...) with (path = '/tmp1') ; INSERT INTO test SELECT ... DROP TABLE test; CREATE TABLE test (...) with (path = '/tmp2'); tEnv.execute(); ``` If we execute the create statements eagerly and the queries lazily we will end up with wrong results.
---------------------------------------------------------------- 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] With regards, Apache Git Services
