+1 (binding)

- Release notes look good
- Source archive signature and checksums good
- Verified there are no binaries in the source archive
- Contents of Maven repo look good
- Built from source/tests pass
- Tag is present in Github
- Fixed web PR as per feedback

Thanks,
Danny

On Wed, Apr 19, 2023 at 8:22 PM Martijn Visser <martijnvis...@apache.org>
wrote:

> +1 (binding)
>
> - Validated hashes
> - Verified signature
> - Verified that no binaries exist in the source archive
> - Build the source with Maven
> - Verified licenses
> - Verified web PRs
>
> On Wed, Apr 19, 2023 at 9:35 AM Martijn Visser <martijnvis...@apache.org>
> wrote:
>
> > Hi all,
> >
> > There was already a blocker ticken op for this, that's
> > https://issues.apache.org/jira/browse/FLINK-31770. There's also an open
> > PR that should resolve it, but is waiting on a reviewer. See
> > https://github.com/apache/flink-connector-jdbc/pull/22
> >
> > I've already done the administration, so this is just fyi :)
> >
> > Best regards,
> >
> > Martijn
> >
> > On Wed, Apr 19, 2023 at 9:17 AM Danny Cranmer <dannycran...@apache.org>
> > wrote:
> >
> >> Thank Benchao for the hotfix. I noticed the CI is also failing/flaky for
> >> the main branch, I have raised a follow-up [1].
> >>
> >> Danny,
> >>
> >> [1] https://issues.apache.org/jira/browse/FLINK-31847
> >>
> >> On Sun, Apr 16, 2023 at 10:57 AM Benchao Li <libenc...@apache.org>
> wrote:
> >>
> >> > And I've filed a hotfix PR[1] to fix the copyright issue.
> >> >
> >> > [1] https://github.com/apache/flink-connector-jdbc/pull/41
> >> >
> >> > Benchao Li <libenc...@apache.org> 于2023年4月16日周日 17:51写道:
> >> >
> >> > > * Verified hashes, signatures (OK)
> >> > > * Download, build and test source code (OK)
> >> > > * Checked files in Nexus staging (OK)
> >> > > * Start a Flink local cluster (1.17.0), run sql-client with jdbc
> >> > > connector jar with Mysql 8.0.28 (OK)
> >> > > * Go through the release note (OK)
> >> > >
> >> > > One thing I spotted is that when I checked the NOTICE file, I found
> >> that
> >> > > we have not updated the copyright year to 2023, I'm not sure whether
> >> we
> >> > > should take this as a blocker issue. If not, my vote is +1.
> >> > >
> >> > > And I left my comments in the PR for website updating.
> >> > >
> >> > > Teoh, Hong <lian...@amazon.co.uk.invalid> 于2023年4月14日周五 15:54写道:
> >> > >
> >> > >> Thanks Danny
> >> > >>
> >> > >> +1 (non-binding)
> >> > >>
> >> > >> * Hashes and Signatures look good
> >> > >> * All required files on dist.apache.org
> >> > >> * Tag is present in Github
> >> > >> * Verified source archive does not contain any binary files
> >> > >> * Source archive builds using maven
> >> > >>
> >> > >>
> >> > >> Cheers,
> >> > >> Hong
> >> > >>
> >> > >>
> >> > >> > On 14 Apr 2023, at 05:21, Elphas Toringepi <elphast...@gmail.com
> >
> >> > >> wrote:
> >> > >> >
> >> > >> > CAUTION: This email originated from outside of the organization.
> Do
> >> > not
> >> > >> click links or open attachments unless you can confirm the sender
> and
> >> > know
> >> > >> the content is safe.
> >> > >> >
> >> > >> >
> >> > >> >
> >> > >> > +1 (non-binding)
> >> > >> >
> >> > >> > * JIRA release notes look good
> >> > >> > * Verified signature and checksum for Apache source
> >> > >> > * Checked source code tag and confirmed Github and Apache source
> >> > >> releases
> >> > >> > are identical
> >> > >> > * Reviewed website pull request and left comment
> >> > >> >
> >> > >> > Kind regards,
> >> > >> > Elphas
> >> > >> >
> >> > >> >
> >> > >> > On Thu, Apr 13, 2023 at 3:42 PM Danny Cranmer <
> >> > dannycran...@apache.org>
> >> > >> > wrote:
> >> > >> >
> >> > >> >> Hi everyone,
> >> > >> >> Please review and vote on the release candidate #1 for the
> version
> >> > >> v3.1.0,
> >> > >> >> as follows:
> >> > >> >> [ ] +1, Approve the release
> >> > >> >> [ ] -1, Do not approve the release (please provide specific
> >> comments)
> >> > >> >>
> >> > >> >> This version supports both Flink 1.16.x and Flink 1.17.x
> >> > >> >>
> >> > >> >> The complete staging area is available for your review, which
> >> > includes:
> >> > >> >> * JIRA release notes [1],
> >> > >> >> * the official Apache source release to be deployed to
> >> > dist.apache.org
> >> > >> >> [2],
> >> > >> >> which are signed with the key with fingerprint
> >> > >> >> 0F79F2AFB2351BC29678544591F9C1EC125FD8DB [3],
> >> > >> >> * all artifacts to be deployed to the Maven Central Repository
> >> [4],
> >> > >> >> * source code tag v3.1.0-rc1 [5],
> >> > >> >> * website pull request listing the new release [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,
> >> > >> >> Danny
> >> > >> >>
> >> > >> >> [1]
> >> > >> >>
> >> > >> >>
> >> > >>
> >> >
> >>
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315522&version=12352678
> >> > >> >> [2]
> >> > >> >>
> >> > >>
> >> >
> >>
> https://dist.apache.org/repos/dist/dev/flink/flink-connector-jdbc-3.1.0-rc1
> >> > >> >> [3] https://dist.apache.org/repos/dist/release/flink/KEYS
> >> > >> >> [4]
> >> > >> >>
> >> > >>
> >> https://repository.apache.org/content/repositories/orgapacheflink-1614/
> >> > >> >> [5]
> >> > >>
> >> https://github.com/apache/flink-connector-jdbc/releases/tag/v3.1.0-rc1
> >> > >> >> [6] https://github.com/apache/flink-web/pull/638
> >> > >> >>
> >> > >>
> >> > >>
> >> > >
> >> > > --
> >> > >
> >> > > Best,
> >> > > Benchao Li
> >> > >
> >> >
> >> >
> >> > --
> >> >
> >> > Best,
> >> > Benchao Li
> >> >
> >>
> >
>

Reply via email to