+1 (binding)

- Checked signature and checksum (OK)
- Checked LICENSE and Copyright year (OK)
- Checked there are no unexpected binary files (OK)
- Build and test from source (testContributorsFileIsSorted and
testMailmapFile in LintTest failed due to not in git repo)
- Reviewed release note, left some comments which can be resolved
after the release

Ruben Q L <rube...@gmail.com> 于2024年4月23日周二 16:57写道:
>
> Thanks Sergey for preparing this release.
> Just a minor clarification: the email subject says "release candidate 1",
> but we are actually voting rc0.
>
> - Release notes: ok (with some minor comments that I have left in the PR)
> - Checksum: ok
> - Signature: ok
> - Diff source release and git repository: ok
> - Build + tests: ok
> - Calcite-based application test suite: ok
>
> +1 (binding)
>
> Best,
> Ruben
>
>
>
> On Tue, Apr 23, 2024 at 12:55 AM Francis Chuang <francischu...@apache.org>
> wrote:
>
> > Thanks for being RM for this release, Sergey!
> >
> > My vote is: +1 (binding)
> >
> > - Verified GPG signature - OK
> > - Verified SHA512 - OK
> > - Diffed source release and git repository - OK
> > - Checked release notes on tag
> > (
> > https://github.com/apache/calcite/blob/calcite-1.37.0-rc0/site/_docs/history.md)
> >
> > - OK
> > - Checked year and versions in NOTICE, README and HOWTO - OK
> > - Ran tests (gradle check) - OK
> > - Spot checked Nexus artifacts - OK
> >
> > Minor issue with the release notes that should be fixed after the
> > release: Some of the contributors are listed as their GitHub usernames,
> > I think it would be better to put in their real names if they are known.
> > These can usually be retrieved from their GitHub profiles.
> >
> > Environment:
> > Eclipse-temurin:8 docker container in WSL2 (Ubuntu 22.04.4) on Windows
> > 11 23h2
> >
> > $ docker version
> > Client:
> >   Cloud integration: v1.0.35+desktop.13
> >   Version:           26.0.0
> >   API version:       1.45
> >   Go version:        go1.21.8
> >   Git commit:        2ae903e
> >   Built:             Wed Mar 20 15:16:45 2024
> >   OS/Arch:           linux/amd64
> >   Context:           default
> >
> > Server: Docker Desktop
> >   Engine:
> >    Version:          26.0.0
> >    API version:      1.45 (minimum version 1.24)
> >    Go version:       go1.21.8
> >    Git commit:       8b79278
> >    Built:            Wed Mar 20 15:18:01 2024
> >    OS/Arch:          linux/amd64
> >    Experimental:     false
> >   containerd:
> >    Version:          1.6.28
> >    GitCommit:        ae07eda36dd25f8a1b98dfbf587313b99c0190bb
> >   runc:
> >    Version:          1.1.12
> >    GitCommit:        v1.1.12-0-g51d5e94
> >   docker-init:
> >    Version:          0.19.0
> >    GitCommit:        de40ad0
> >
> > $ gradle -v
> >
> > ------------------------------------------------------------
> > Gradle 7.6.1
> > ------------------------------------------------------------
> >
> > Build time:   2023-02-24 13:54:42 UTC
> > Revision:     3905fe8ac072bbd925c70ddbddddf4463341f4b4
> >
> > Kotlin:       1.7.10
> > Groovy:       3.0.13
> > Ant:          Apache Ant(TM) version 1.10.11 compiled on July 10 2021
> > JVM:          1.8.0_402 (Temurin 25.402-b06)
> > OS:           Linux 5.15.146.1-microsoft-standard-WSL2 amd64
> >
> > $ java -version
> > openjdk version "1.8.0_402"
> > OpenJDK Runtime Environment (Temurin)(build 1.8.0_402-b06)
> > OpenJDK 64-Bit Server VM (Temurin)(build 25.402-b06, mixed mode)
> >
> > Francis
> >
> > On 23/04/2024 9:18 am, Sergey Nuyanzin wrote:
> > > Hi all,
> > >
> > > I have created a build for Apache Calcite 1.37.0, release
> > > candidate 0.
> > >
> > > Thanks to everyone who has contributed to this release.
> > >
> > > You can read the release notes here:
> > >
> > https://github.com/apache/calcite/blob/calcite-1.37.0-rc0/site/_docs/history.md
> > >
> > > The commit to be voted upon:
> > >
> > https://gitbox.apache.org/repos/asf?p=calcite.git;a=commit;h=53fd905eb4c730b4bedd49fa6569d5b0de47f021
> > >
> > > Its hash is 53fd905eb4c730b4bedd49fa6569d5b0de47f021
> > >
> > > Tag:
> > > https://github.com/apache/calcite/tree/calcite-1.37.0-rc0
> > >
> > > The artifacts to be voted on are located here:
> > > https://dist.apache.org/repos/dist/dev/calcite/apache-calcite-1.37.0-rc0
> > > (revision 68720)
> > >
> > > The hashes of the artifacts are as follows:
> > >
> > b044fb94fd60710142edfe348ff20bbdc7e43f3d70a68b6520b65dc8fb2fe53f0ecf6b5d03b199b3212e9064a7dd04fb3995a9cd09cb8864a7bfdb5617feefbb
> > > *apache-calcite-1.37.0-src.tar.gz
> > >
> > > A staged Maven repository is available for review at:
> > >
> > https://repository.apache.org/content/repositories/orgapachecalcite-1225/org/apache/calcite/
> > >
> > > Release artifacts are signed with the following key:
> > > https://people.apache.org/keys/committer/snuyanzin.asc
> > > https://www.apache.org/dist/calcite/KEYS
> > >
> > > To create the jars and test Apache Calcite: "gradle build"
> > > (requires an appropriate Gradle/JDK installation)
> > >
> > > Please vote on releasing this package as Apache Calcite 1.37.0.
> > >
> > > The vote is open for the next 72 hours and passes if a majority of at
> > > least three +1 PMC votes are cast.
> > >
> > > [ ] +1 Release this package as Apache Calcite 1.37.0
> > > [ ]  0 I don't feel strongly about it, but I'm okay with the release
> > > [ ] -1 Do not release this package because...
> > >
> > > Here is my vote:
> > >
> > > +1 (binding)
> > >
> >



-- 

Best,
Benchao Li

Reply via email to