echauchot commented on pull request #15156: URL: https://github.com/apache/flink/pull/15156#issuecomment-848561600
Hi @AHeise, thanks for your detailed comment. Merging my 2 PRs to 1.13 looks fine to me, I think backporting to 1.12 might me considered as well as Amazon EMR is still 1.12 based (latest is 1.12.1) and they are likely to move to latest 1.12 in short term releases of EMR. For the context, what I'm doing is a benchmark comparing Flink Dataset/Datstream/SQL and other technologies such as Apache Beam. So I'd like to have a working DataSet/DataStream pipeline in addition to my existing working SQL one (using blink planner based on DataStream and BatchTableEnvironment). And I guess there will be some users that will want to stick to DataSet/DataStream if they have existing pipelines that they do not want to re-code but just migrate to next flink version. -- 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
