Ubuntu 20.04.4 LTS, jdk1.8.0_261, Gradle wrapper, Gradle 7.4.2 * Checked signatures and checksums OK * Went over release note OK (left some comments in the draft PR) * Built from git tag and run tests (./gradlew clean build) OK * Built from source artifacts and run unit tests + slow tests OK * Checked diff between git repo and release sources KO
Comparing the contents between the git repo and the release sources I found that the (RC0) sources contain two (empty) directories (i.e., arrow, .mvn) that shouldn't be there. I guess there was some kind of stale state in the calcite directory when preparing the RC. The command that I used to compare the sources can be found below. $ diff -qr apache-calcite-1.31.0-src apache-calcite-1.31.0-git Only in apache-calcite-1.31.0-src: arrow Only in apache-calcite-1.31.0-git: .git Only in apache-calcite-1.31.0-git/gradle: wrapper Only in apache-calcite-1.31.0-git: gradlew Only in apache-calcite-1.31.0-git: gradlew.bat Only in apache-calcite-1.31.0-src: licenses Only in apache-calcite-1.31.0-src: .mvn Only in apache-calcite-1.31.0-git/site: fonts The rest of the differences look normal and have been discussed previously. 0 (binding) Best, Stamatis On Mon, Jul 25, 2022 at 2:46 AM Francis Chuang <[email protected]> wrote: > Thanks for being RM for this release, Andrei. > > My vote is: +1 (binding) > > - Verified GPG signature - OK > - Verified SHA512 - OK > - Checked release notes on tag > ( > https://github.com/apache/calcite/blob/calcite-1.31.0-rc0/site/_docs/history.md) > > - OK > - Ran tests (gradle check) - OK > - Spot checked Nexus artifacts - OK > > Environment: > Eclipse-temurin:17-jammy docker container in WSL2 (Ubuntu 20.04) on > Windows 10 21h2 > > > docker version > Client: Docker Engine - Community > Cloud integration: v1.0.24 > Version: 20.10.17 > API version: 1.41 > Go version: go1.17.11 > Git commit: 100c701 > Built: Mon Jun 6 23:03:17 2022 > OS/Arch: linux/amd64 > Context: default > Experimental: true > > Server: Docker Desktop > Engine: > Version: 20.10.17 > API version: 1.41 (minimum version 1.12) > Go version: go1.17.11 > Git commit: a89b842 > Built: Mon Jun 6 23:01:23 2022 > OS/Arch: linux/amd64 > Experimental: false > containerd: > Version: 1.6.6 > GitCommit: 10c12954828e7c7c9b6e0ea9b0c02b01407d3ae1 > runc: > Version: 1.1.2 > GitCommit: v1.1.2-0-ga916309 > docker-init: > Version: 0.19.0 > GitCommit: de40ad0 > > > gradle -v > > ------------------------------------------------------------ > Gradle 7.4.2 > ------------------------------------------------------------ > > Build time: 2022-03-31 15:25:29 UTC > Revision: 540473b8118064efcc264694cbcaa4b677f61041 > > Kotlin: 1.5.31 > Groovy: 3.0.9 > Ant: Apache Ant(TM) version 1.10.11 compiled on July 10 2021 > JVM: 17.0.3 (Eclipse Adoptium 17.0.3+7) > OS: Linux 5.10.102.1-microsoft-standard-WSL2 amd64 > > > java -version > openjdk version "17.0.3" 2022-04-19 > OpenJDK Runtime Environment Temurin-17.0.3+7 (build 17.0.3+7) > OpenJDK 64-Bit Server VM Temurin-17.0.3+7 (build 17.0.3+7, mixed mode, > sharing) > > Regarding the java aggregate docs pointed out by Benchao: I think it > should be fine after they are automatically deployed when the release is > finalized. The previous version's website was manually deployed, so I > think there was a small error in the deployment. > > Francis > > On 23/07/2022 1:07 pm, Andrei Sereda wrote: > > Hi all, > > > > I have created a build for Apache Calcite 1.31.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.31.0-rc0/site/_docs/history.md > > > > The commit to be voted upon: > > > https://gitbox.apache.org/repos/asf?p=calcite.git;a=commit;h=c416e109460d8e439078206a48057b504b6bb08b > > > > Its hash is c416e109460d8e439078206a48057b504b6bb08b > > > > Tag: > > > https://gitbox.apache.org/repos/asf?p=calcite.git;a=tag;h=refs/tags/calcite-1.31.0-rc0 > > > > The artifacts to be voted on are located here: > > https://dist.apache.org/repos/dist/dev/calcite/apache-calcite-1.31.0-rc0 > > (revision 55918) > > > > The hashes of the artifacts are as follows: > > > 527e699857958b9d98e733059bd67ca6fc42e75238074dec65cc86caa8f5ae176e269d9ac478754ad206ea2336289a8f40f37ee86818130b37b53a53ee79bb2c > > *apache-calcite-1.31.0-src.tar.gz > > > > A staged Maven repository is available for review at: > > > https://repository.apache.org/content/repositories/orgapachecalcite-1166/org/apache/calcite/ > > > > Release artifacts are signed with the following key: > > https://people.apache.org/keys/committer/sereda.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.31.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.31.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 (non-binding) > > >
