Hello!  +1 (non-binding)

I've validated the source for the RC1:
flink-connector-opensearch-1.0.1-src.tgz
* The sha512 checksum is OK.
* The source file is signed correctly.
* The signature A5F3BCE4CBE993573EC5966A65321B8382B219AF is found in the
KEYS file, and on https://keys.openpgp.org
* The source file is consistent with the Github tag v1.0.1-rc1, which
corresponds to commit c52dbf4fc9c473592479a6c4fc6b2b5227699737
   - The files explicitly excluded by create_pristine_sources (such as
.gitignore and the submodule tools/releasing/shared) are not present.
* Has a LICENSE file and a NOTICE file.  The sql-connector has a
NOTICE file for bundled artifacts.
* Does not contain any compiled binaries.

* The sources can be compiled and tests pass with flink.version 1.17.0 and
flink.version 1.16.1

* Nexus has three staged artifact ids for 1.0.1-1.16 and 1.0.1-1.17
 - flink-connector-opensearch-parent (only .pom)
 - flink-connector-opensearch (.jar, -sources.jar, -javadoc.jar, -tests.jar
and .pom)
 - flink-sql-connector-opensearch (.jar, -sources.jar and .pom)
* All 18 files have been signed with the same key as above, and have
correct sha1 and md5 checksums.

I didn't run any additional smoke tests other than the integration test
cases.

A couple minor points, but nothing that would block this release.

- like the other connectors I've checked, flink.version in the parent pom
is set to 1.16.0 even for 1.17 artifacts, which might be confusing.
- the NOTICE files have the wrong year.
- unlike other connectors, flink-connector-opensearch publishes the
-tests.jar classifier to nexus.  Is this desired?
- The sql-connector PackagingITCase test fails when using
`-Prelease,docs-and-source`, but otherwise works as intended.

All my best and thanks for the release.

Ryan

On Thu, Apr 13, 2023 at 3:39 PM Andrey Redko <drr...@gmail.com> wrote:

> +1 (non-binding), thanks Martijn!
>
> Best Regards,
>     Andriy Redko
>
> On Thu, Apr 13, 2023, 8:54 AM Martijn Visser <martijnvis...@apache.org>
> wrote:
>
> > Hi everyone,
> > Please review and vote on the release candidate #1 for the version 1.0.1,
> > 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 to be deployed to dist.apache.org
> > [2],
> > which are signed with the key with fingerprint
> > A5F3BCE4CBE993573EC5966A65321B8382B219AF [3],
> > * all artifacts to be deployed to the Maven Central Repository [4],
> > * source code tag v1.0.1-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,
> > Release Manager
> >
> > [1]
> >
> >
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315522&version=12352686
> > [2]
> >
> >
> https://dist.apache.org/repos/dist/dev/flink/flink-connector-opensearch-1.0.1-rc1
> > [3] https://dist.apache.org/repos/dist/release/flink/KEYS
> > [4]
> > https://repository.apache.org/content/repositories/orgapacheflink-1612/
> > [5] https://github.com/apache/flink-connector-
> > <connector>/releases/tag/v1.0.1-rc1
> > [6] https://github.com/apache/flink-web/pull/636
> >
>

Reply via email to