+1 Forgot to mention I am also pro removal of 1.7 support, but let's wait a bit to see if someone differs.
On Tue, Feb 18, 2020 at 6:46 PM Thomas Weise <[email protected]> wrote: > +1 for dropping Flink 1.7 support > > I agree that we should avoid the SNAPSHOT dependency. > > Generally I think that the version support should remain based on user > feedback and independent of the Flink release policy. If there are Beam > users that require an older Flink version and it is difficult to upgrade, > then we should consider that. > > > On Tue, Feb 18, 2020 at 8:57 AM Ismaël Mejía <[email protected]> wrote: > >> Just to be sure, you mean Flink 1.11.0-SNAPSHOT ONLY in the next branch >> dependency? >> We should not have any SNAPSHOT dependency from other projects in Beam. >> >> On Tue, Feb 18, 2020 at 5:34 PM <[email protected]> wrote: >> >>> HiCr Jincheng, >>> I think there should be a "process" for this. I would propose to: >>> a) create new branch with support for new (snapshot) flink - currently >>> that would mean flink 1.11 >>> b) as part of this brach drop support for all version up to N - 3 >>> I think th!t that dropping all versions and adding new version should be >>> atomic, otherwise we risk we release beam version with less than three >>> supprted flink versions. >>> I'd suggest to start with the 1.10 branch support, include the drop of >>> 1.7 into that branch. Once 1.10 gets merged, we should create 1.11 with >>> snapshot dependency to be able to keep up with the release cadence of flink. >>> WDYT? >>> Jan >>> >>> Dne 18. 2. 2020 15:34 napsal uživatel jincheng sun < >>> [email protected]>: >>> >>> Hi folks, >>> >>> Apache Flink 1.10 has completed the release announcement [1]n Then we >>> would like to add Flink 1.10 build target and make Flink Runner compatible >>> with Flink 1.10 [2]. So, I would suggest that at most three versions of >>> Flink runner for Apache Beam community according to the update Policy of >>> Apache Flink releases [3] , i.e. I think it's better to maintain the three >>> versions of 1.8/1.9/1.10 after add Flink 1.10 build target to Flink runner. >>> >>> The current existence of Flink runner 1.7 will affect the upgrade of >>> Flink runner 1.8x and 1.9x due to the code of Flink 1.7 is too old, more >>> detail can be found in [4]. So, we need to drop the support of Flink >>> runner 1.7 as soon as possible. >>> >>> This discussion also CC to @User, due to the change will affect our >>> users. And I would appreciate it if you could review the PR [5]. >>> >>> Welcome any feedback! >>> >>> Best, >>> Jincheng >>> >>> [1] >>> http://apache-flink-mailing-list-archive.1008284.n3.nabble.com/ANNOUNCE-Apache-Flin+-1-10-0-released-td37564.html >>> <http://apache-flink-mailing-list-archive.1008284.n3.nabble.com/ANNOUNCE-Apache-Flink-1-10-0-released-td37564.html> >>> [2] https://issues.apache.org/jira/browse/BEAM-9295 >>> [3] >>> https://flink.apache.org/downloads.html#update-policy-for-old-releases >>> [4] https://issues.apache.org/jira/browse/BEAM-9299 >>> [5] https://github.com/apache/beam/pull/10884 >>> >>> >>>
