+1 (non-binding) - verify signatures and checksums - no binaries found in source archive - build from source code - verify python wheel package contents - pip install apache-flink-libraries and apache-flink wheel packages - thread mode works as expected in Python DataStream API - the Python DataStream Window works as expected - the Python Sideoutput works as expected
Best, Xingbo Lijie Wang <wangdachui9...@gmail.com> 于2022年10月24日周一 16:09写道： > +1 (non-binding) for this candidate > > * Built from the source code. > * Verified the signature and checksum > * Ran both streaming/batch jobs on yarn cluster > * The new speculative execution works as expected > > Best, > Lijie > > Yun Tang <myas...@live.com> 于2022年10月22日周六 15:20写道： > > > +1 (non-binding) > > > > > > * build from the source code. > > * verified the signature of the release binaries. > > * launched a standalone cluster with examples for both > streaming/batch > > execution modes. > > * checked rocksdb's log would exist under Flink's default log folder. > > * reviewed the flink-web PR > > > > Best > > Yun Tang > > ________________________________ > > From: Xintong Song <tonysong...@gmail.com> > > Sent: Friday, October 21, 2022 18:29 > > To: dev@flink.apache.org <dev@flink.apache.org> > > Subject: Re: [VOTE] Release 1.16.0, release candidate #2 > > > > +1 (binding) for this release candidate > > > > - verified checksum & signature > > - build from source > > - tried example jobs with standalone cluster, web ui & logs look good > > - the new completed jobs & history server features work as expected > > - the new hybrid shuffle mode works as expected > > > > Best, > > > > Xintong > > > > > > > > On Fri, Oct 21, 2022 at 11:09 AM godfrey he <godfre...@gmail.com> wrote: > > > > > +1 to make 1.16.0 released as soon as possible, > > > it's been more than two months since feature freeze, > > > 1.17 already starts kicking off. We can fix the critical bugs in > 1.16.1. > > > > > > Best, > > > Godfrey > > > > > > Xintong Song <tonysong...@gmail.com> 于2022年10月21日周五 09:57写道： > > > > > > > > BTW, missing 1.16.0 is probably not that bad. From my experience, the > > > x.y.0 > > > > releases are usually considered unstable and are mainly for trying > out > > > > purposes. Most users do not upgrade to the new version in production > > > until > > > > the x.y.1/2 releases, which are considered more stable. As this > bug-fix > > > is > > > > making 1.16.1 anyway, it barely misses anything. > > > > > > > > Best, > > > > > > > > Xintong > > > > > > > > > > > > > > > > On Fri, Oct 21, 2022 at 5:10 AM Danny Cranmer < > dannycran...@apache.org > > > > > > > wrote: > > > > > > > > > We had a similar situation for Flink 1.15.1 where a non-regression > > > > > "critical" bug was impacting a connector [1]. We decided to not > block > > > the > > > > > release to address this issue. Based on this, I am inclined to > agree > > > with > > > > > Martijn and move forward with the release. This bug is not marked > as > > a > > > > > "blocker" and we should respect that. > > > > > > > > > > I am happy to manage a 1.15.3 or 1.16.1 release follow up to > address > > > these > > > > > concerns. > > > > > > > > > > [1] > https://lists.apache.org/thread/8hxz5n6x2v4c6v8z33hdz51h1bhn768d > > > > > > > > > > Thanks, > > > > > Danny > > > > > > > > > > On Thu, Oct 20, 2022 at 5:01 PM Martijn Visser < > > > martijnvis...@apache.org> > > > > > wrote: > > > > > > > > > > > Taking in this fix would require us to cancel RC2 and create > > another > > > > > > release candidate. We are already long-overdue on the Flink 1.16 > > > release. > > > > > > Given that 1.15.3 is not yet released, it can't be a regression > > > compared > > > > > to > > > > > > the current situation of 1.15.2. The Flink Delta connector is not > > > part of > > > > > > the ASF Flink community so it can't be considered a blocker from > a > > > ASF > > > > > > Flink community perspective. > > > > > > > > > > > > I do understand the pain. I'm curious what others think if this > is > > > worthy > > > > > > of cancelling the release candidate. > > > > > > > > > > > > Thanks, Martijn > > > > > > > > > > > > On Thu, Oct 20, 2022 at 4:54 PM Krzysztof Chmielewski < > > > > > > krzysiek.chmielew...@gmail.com> wrote: > > > > > > > > > > > > > Thank you all for response, > > > > > > > however i think you may miss a bigger context regarding those 3 > > > > > tickets. > > > > > > > > > > > > > > Those 3 tickets [29509, 29512, 29627] are part of a bigger > thing. > > > They > > > > > > are > > > > > > > fixing 1.15 Sink V2 issue, where Task manager will not start > > after > > > > > > recovery > > > > > > > for Sink topology with Global Committer. The problem was > > described > > > by > > > > > me > > > > > > in > > > > > > > this thread [1]. We need all three to fix the problem. > > > > > > > > > > > > > > All three tickets were merged into 1.15 release branch and will > > be > > > > > > included > > > > > > > in 1.15.3 probably, however 1.16 will be missing one fix > (29627). > > > > > > > In other words, there will be a regression between 1.16 and > > 1.15.3. > > > > > > > > > > > > > > Additionally for now this issue is blocking Flink migration for > > > Delta > > > > > > > connector [2]. > > > > > > > We need to migrate because Flink 1.14 has another Sink problem > > with > > > > > data > > > > > > > loss during Sink Recovery with Global Committer [3] and this > one > > > most > > > > > > > likely will not be fixed since 1.14 support is ending. > > > > > > > > > > > > > > Forgive my if I'm wrong but what do you mean by " we won't > block > > > 1.16.0 > > > > > > on > > > > > > > this." Fix is merged so couldn't we just cherry pick 1.16 merge > > > commit > > > > > to > > > > > > > 1.16.0's RC2? > > > > > > > > > > > > > > [1] > > > https://lists.apache.org/thread/otscy199g1l9t3llvo8s2slntyn2r1jc > > > > > > > [2] https://github.com/delta-io/connectors/tree/master/flink > > > > > > > [3] https://issues.apache.org/jira/browse/FLINK-29589 > > > > > > > > > > > > > > Regards, > > > > > > > Krzysztof Chmielewski > > > > > > > > > > > > > > czw., 20 paź 2022 o 16:13 Xingbo Huang <h...@apache.org> > > > napisał(a): > > > > > > > > > > > > > > > Hi Krzysztof, > > > > > > > > > > > > > > > > When I was building rc2, I tried to search whether issues > with > > > `fix > > > > > > > > version` of 1.16.0 have not been closed. > > > > > > > > https://issues.apache.org/jira/browse/FLINK-29627 was missed > > > because > > > > > > the > > > > > > > > `fix version` was not marked. I agree with Martijn and > Xintong > > > that > > > > > we > > > > > > > > won't block 1.16.0 on this. > > > > > > > > > > > > > > > > Best, > > > > > > > > Xingbo > > > > > > > > > > > > > > > > Xintong Song <tonysong...@gmail.com> 于2022年10月20日周四 18:23写道： > > > > > > > > > > > > > > > > > Hi Krzysztof, > > > > > > > > > > > > > > > > > > FLINK-29627 is merged after rc2 being created, that's why > it > > > > > doesn't > > > > > > > > appear > > > > > > > > > in the change list. See the commit history of rc2 [1]. > > > > > > > > > > > > > > > > > > It's unfortunate this fix didn't make the 1.16.0 release > (if > > > rc2 is > > > > > > > > > approved). However, I agree with Martijn that we should not > > > further > > > > > > > block > > > > > > > > > 1.16.0 on this. If there's no other blockers discovered in > > the > > > rc2, > > > > > > the > > > > > > > > fix > > > > > > > > > of FLINK-29627 will be shipped in 1.16.1. > > > > > > > > > > > > > > > > > > Best, > > > > > > > > > > > > > > > > > > Xintong > > > > > > > > > > > > > > > > > > > > > > > > > > > [1] > > https://github.com/apache/flink/commits/release-1.16.0-rc2 > > > > > > > > > > > > > > > > > > On Thu, Oct 20, 2022 at 6:16 PM Krzysztof Chmielewski < > > > > > > > > > krzysiek.chmielew...@gmail.com> wrote: > > > > > > > > > > > > > > > > > > > Thanks Martijn, > > > > > > > > > > just to clarify from my end, > > > > > > > > > > > > > > > > > > > > All three tickets, [1] [2] [3] are fixed and merged to > > 1.16 > > > > > branch > > > > > > > > > > already. I just noticed that one of them [3] is not on > > > included > > > > > in > > > > > > > the > > > > > > > > > > change list. Hence my email. > > > > > > > > > > > > > > > > > > > > [1] https://issues.apache.org/jira/browse/FLINK-29509 > > > > > > > > > > [2] https://issues.apache.org/jira/browse/FLINK-29512 > > > > > > > > > > [3] https://issues.apache.org/jira/browse/FLINK-29627 > > > > > > > > > > > > > > > > > > > > Regards, > > > > > > > > > > Krzysztof Chmielewski > > > > > > > > > > > > > > > > > > > > czw., 20 paź 2022 o 11:51 Martijn Visser < > > > > > martijnvis...@apache.org > > > > > > > > > > > > > > > > > napisał(a): > > > > > > > > > > > > > > > > > > > > > Hi Krzysztof, > > > > > > > > > > > > > > > > > > > > > > Given that this issue already exists in previous Flink > > > > > versions, > > > > > > I > > > > > > > > > don't > > > > > > > > > > > think it's a blocker for 1.16. We should get it fixed > > (all > > > of > > > > > the > > > > > > > > > > tickets) > > > > > > > > > > > so it will be addressed in a new Flink 1.15 version, in > > > Flink > > > > > > > 1.16.1 > > > > > > > > > and > > > > > > > > > > of > > > > > > > > > > > course Flink 1.17. > > > > > > > > > > > > > > > > > > > > > > Best regards, > > > > > > > > > > > > > > > > > > > > > > Martijn > > > > > > > > > > > > > > > > > > > > > > On Thu, Oct 20, 2022 at 11:47 AM Krzysztof Chmielewski > < > > > > > > > > > > > krzysiek.chmielew...@gmail.com> wrote: > > > > > > > > > > > > > > > > > > > > > > > Hi, > > > > > > > > > > > > I would like to ask about [1] ticket with PR [2]. It > > was > > > > > merged > > > > > > > to > > > > > > > > > 1.16 > > > > > > > > > > > > release branch today but I do not see it on the > change > > > list. > > > > > > > > > > > > > > > > > > > > > > > > It is closely related to [3] and [4] that are on the > > > change > > > > > > list. > > > > > > > > > > However > > > > > > > > > > > > to fully fix Sink architecture issue we need all 3 > > > tickets > > > > > [1], > > > > > > > [3] > > > > > > > > > and > > > > > > > > > > > [4] > > > > > > > > > > > > > > > > > > > > > > > > [1] > https://issues.apache.org/jira/browse/FLINK-29627 > > > > > > > > > > > > [2] https://github.com/apache/flink/pull/21115 > > > > > > > > > > > > [3] > https://issues.apache.org/jira/browse/FLINK-29509 > > > > > > > > > > > > [4] > https://issues.apache.org/jira/browse/FLINK-29512 > > > > > > > > > > > > > > > > > > > > > > > > Regards, > > > > > > > > > > > > Krzysztof Chmielewski > > > > > > > > > > > > > > > > > > > > > > > > czw., 20 paź 2022 o 11:21 Xingbo Huang < > h...@apache.org > > > > > > > > > > > napisał(a): > > > > > > > > > > > > > > > > > > > > > > > > > Hi everyone, > > > > > > > > > > > > > > > > > > > > > > > > > > Please review and vote on the release candidate #2 > > for > > > the > > > > > > > > version > > > > > > > > > > > > 1.16.0, > > > > > > > > > > > > > as follows: > > > > > > > > > > > > > [ ] +1, Approve the release > > > > > > > > > > > > > [ ] -1, Do not approve the release (please provide > > > specific > > > > > > > > > comments) > > > > > > > > > > > > > > > > > > > > > > > > > > The complete staging area is available for your > > review, > > > > > which > > > > > > > > > > includes: > > > > > > > > > > > > > * JIRA release notes [1], > > > > > > > > > > > > > * the official Apache source release and binary > > > convenience > > > > > > > > > releases > > > > > > > > > > to > > > > > > > > > > > > be > > > > > > > > > > > > > deployed to dist.apache.org [2], which are signed > > > with the > > > > > > key > > > > > > > > > with > > > > > > > > > > > > > fingerprint 3C2C9FFB59DF9F3E [3], > > > > > > > > > > > > > * all artifacts to be deployed to the Maven Central > > > > > > Repository > > > > > > > > [4], > > > > > > > > > > > > > * source code tag "release-1.16.0-rc2" [5], > > > > > > > > > > > > > * website pull request listing the new release and > > > adding > > > > > > > > > > announcement > > > > > > > > > > > > blog > > > > > > > > > > > > > post [6]. > > > > > > > > > > > > > > > > > > > > > > > > > > The vote will be open for at least 72 hours. It is > > > adopted > > > > > by > > > > > > > > > > majority > > > > > > > > > > > > > approval, with at least 3 PMC affirmative votes. > > > > > > > > > > > > > > > > > > > > > > > > > > Thanks, > > > > > > > > > > > > > Chesnay, Martijn, Godfrey & Xingbo > > > > > > > > > > > > > > > > > > > > > > > > > > [1] > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315522&version=12351275 > > > > > > > > > > > > > [2] > > > > > > > > > https://dist.apache.org/repos/dist/dev/flink/flink-1.16.0-rc2 > > > > > > > > > > > > > [3] > > > https://dist.apache.org/repos/dist/release/flink/KEYS > > > > > > > > > > > > > [4] > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://repository.apache.org/content/repositories/orgapacheflink-1541 > > > > > > > > > > > > > [5] > > > > > https://github.com/apache/flink/tree/release-1.16.0-rc2 > > > > > > > > > > > > > [6] https://github.com/apache/flink-web/pull/574 > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
