+1 (non-binding) after verifying as follows:

- verified gpg signature: OK

$ curl "https://downloads.apache.org/calcite/KEYS"; | gpg --import

$ gpg --verify apache-calcite-avatica-1.25.0-src.tar.gz.asc
apache-calcite-avatica-1.25.0-src.tar.gz

- verified package checksum: OK

$ diff <(cat apache-calcite-avatica-1.25.0-src.tar.gz.sha512) <(shasum -a
512 apache-calcite-avatica-1.25.0-src.tar.gz)

- verified gradle build from sources: OK

$ cp ~/git-apache/calcite-avatica/gradlew .

$ cp -r ~/git-apache/calcite-avatica/gradle/wrapper gradle

$ ./gradlew build -Prelease -PskipSign

$ docker-compose run test

- building commit and running tests: OK

$ ./gradlew build -Prelease -PskipSign

$ docker-compose run test

(checked CI on github for the commit)

- checked release notes: OK

- checked few modules in Nexus: OK

- checking difference in folder: OK

$ diff -qr . ~/git-apache/calcite-avatica | grep -v gradle | grep -v idea

(nothing worth mentioning)

-- Environment used:

$ sw_vers

ProductName: macOS

ProductVersion: 12.6.2

BuildVersion: 21G320

$ ./gradlew -version

------------------------------------------------------------

Gradle 8.5

------------------------------------------------------------

Build time:   2023-11-29 14:08:57 UTC

Revision:     28aca86a7180baa17117e0e5ba01d8ea9feca598

Kotlin:       1.9.20

Groovy:       3.0.17

Ant:          Apache Ant(TM) version 1.10.13 compiled on January 4 2023

JVM:          1.8.0_352 (Azul Systems, Inc. 25.352-b08)

OS:           Mac OS X 12.6.2 aarch64

$ java -version

openjdk version "1.8.0_352"

OpenJDK Runtime Environment (Zulu 8.66.0.15-CA-macos-aarch64) (build
1.8.0_352-b08)

OpenJDK 64-Bit Server VM (Zulu 8.66.0.15-CA-macos-aarch64) (build
25.352-b08, mixed mode)

Best regards,

Alessandro

On Thu, 4 Apr 2024 at 09:25, Stamatis Zampetakis <zabe...@gmail.com> wrote:

> Ubuntu 20.04.6 LTS, jdk1.8.0_261, Gradle wrapper, Gradle 8.1.1
>
>  * Checked signatures and checksums OK
>  * Checked diff between repo and artifacts OK
>  * Went over release note OK (It mentions two Gradle versions 8.4 and 8.5)
>  * Checked README, NOTICE, LICENSE OK (minor fixes in LICENSE from
> Calcite should be ported to Avatica)
>  * All source files have ASF headers OK (grep -RiL "Licensed to the
> Apache Software Foundation")
>  * No unexpected binary files OK (find . -type f -exec file {} \; |
> grep -v text)
>  * Checked LICENSE, NOTICE, signature, and checksum for
> avatica-core-1.25.0.jar in nexus OK
>  * Built from git tag and run tests (./gradlew build) OK
>  * Built from source artifacts and run tests (gradle build) OK
>
> +1 (binding)
>
> Best,
> Stamatis
>
>
> On Wed, Apr 3, 2024 at 11:36 PM Francis Chuang <francischu...@apache.org>
> wrote:
> >
> > Opps, good catch! Please ignore the links to the
> > calcite-avatica-site-preview repo. Those are generated by the
> > asf-release plugin, but we don't use any preview repos for Calcite.
> >
> > On 3/04/2024 9:41 pm, Benchao Li wrote:
> > > +1 (binding)
> > >
> > > - checked signature and checksum [OK]
> > > - checked copyright year in notice [OK]
> > > - downloaded src, make sure no unexpected files [OK]
> > > - compiled and tested from source [OK]
> > > - reviewed release note [OK]
> > > - checked a few files in Nexus for signature and checksum [OK]
> > > - downloaded javadoc jar, make sure it's in English [OK]
> > >
> > > One minor comment is that those gitbox urls seem not valid.
> > >
> > > Francis Chuang <francischu...@apache.org> 于2024年4月2日周二 11:40写道:
> > >>
> > >> Hi all,
> > >>
> > >> I have created a build for Apache Calcite Avatica 1.25.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-avatica/blob/avatica-1.25.0-rc0/site/_docs/history.md
> > >>
> > >> The commit to be voted upon:
> > >>
> https://gitbox.apache.org/repos/asf?p=calcite-avatica.git;a=commit;h=62b0fdd3f4e0173d8492eb0e203056f0938dd6f5
> > >>
> > >> Its hash is 62b0fdd3f4e0173d8492eb0e203056f0938dd6f5
> > >>
> > >> Tag:
> > >> https://github.com/apache/calcite-avatica/tree/avatica-1.25.0-rc0
> > >>
> > >> The artifacts to be voted on are located here:
> > >>
> https://dist.apache.org/repos/dist/dev/calcite/apache-calcite-avatica-1.25.0-rc0
> > >>
> > >> RAT report:
> > >>
> https://gitbox.apache.org/repos/asf/calcite-avatica-site-preview.git/avatica/rat/rat-report.txt
> > >>
> > >> Site preview is here:
> > >>
> https://gitbox.apache.org/repos/asf/calcite-avatica-site-preview.git/avatica/
> > >>
> > >> JavaDoc API preview is here:
> > >>
> https://gitbox.apache.org/repos/asf/calcite-avatica-site-preview.git/avatica/api
> > >>
> > >> The hashes of the artifacts are as follows:
> > >>
> 0daa1da33ca0712cb3a4918260081e707c336e06eaacee87962a8ff914f377e8ad21d5fd24490612dbca7d56b83e9d062c39b5cd51701564cf8cba7840ed4806
> > >> *apache-calcite-avatica-1.25.0-src.tar.gz
> > >>
> > >> A staged Maven repository is available for review at:
> > >>
> https://repository.apache.org/content/repositories/orgapachecalcite-1223/org/apache/calcite/
> > >>
> > >> Release artifacts are signed with the following key:
> > >> https://people.apache.org/keys/committer/francischuang.asc
> > >> https://www.apache.org/dist/calcite/KEYS
> > >>
> > >> To create the jars and test Apache Calcite Avatica: "gradle build
> > >> -Prelease -PskipSign".
> > >>
> > >> If you do not have a Java/Gradle environment available, you can run
> the
> > >> tests using docker. To do so, install docker and docker-compose, then
> > >> run "docker compose run test" from the root of the directory.
> > >>
> > >> Please vote on releasing this package as Apache Calcite Avatica
> 1.25.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 Avatica 1.25.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)
> > >>
> > >> Francis
> > >
> > >
> > >
>

Reply via email to