+1 (non-binding) * Downloaded artifacts * Verified checksums/GPG signatures * Compared checkout with provided sources * Verified pom file versions * Went over NOTICE file/pom files changes without finding anything suspicious * Build Flink from sources * Deployed standalone session cluster and ran WordCount example in batch and streaming: Nothing suspicious in log files found
On Mon, Oct 24, 2022 at 3:51 PM Sergey Nuyanzin <snuyan...@gmail.com> wrote: > +1 (non-binding) > > - checked hashes and signatures > - built from sources > - started cluster, ran different simple jobs > - checked sql client > > > On Mon, Oct 24, 2022 at 3:14 PM Leonard Xu <xbjt...@gmail.com> wrote: > > > +1 (non-binding) > > > > - verified signatures and hashsums > > - built from source code succeeded > > - checked all dependency artifacts are 1.16 > > - started a cluster, ran a wordcount job, the result is expected, no > > suspicious log output > > - started SQL Gateway, tested several rest APIs, the SQL query results > are > > expected > > > > Best, > > Leonard Xu > > > > > > > 2022年10月24日 下午8:49，Xingbo Huang <hxbks...@gmail.com> 写道： > > > > > > +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 > > >>>>>>>>>>>>>> > > >>>>>>>>>>>>> > > >>>>>>>>>>>> > > >>>>>>>>>>> > > >>>>>>>>>> > > >>>>>>>>> > > >>>>>>>> > > >>>>>>> > > >>>>>> > > >>>> > > >>> > > >> > > > > > > -- > Best regards, > Sergey >
