Hi Alex that's correct, the header is not correct in the file introduced by #4823.
I cancel thiis RC to fix that. Regards JB On Tue, Sep 22, 2026 at 7:07 PM Alex Dutra <[email protected]> wrote: > -1 (binding) > > I verified the following for Apache Polaris 1.8.0-rc1: > > - Automated checks (verify-release.sh): OK > - License report validation (./gradlew generateLicenseReport): OK > - CHANGELOG/version.txt: OK > - Smoke tests: > - Binary distribution: OK > - Source distribution: OK > - Helm chart: OK > > However, I found a licensing issue introduced by PR [4823] that I > believe warrants a -1: it created a new file, > > tools/config-docs/generator/src/test/java/tests/properties/NestedProps.java, > with an incorrect Dremio license header. Since this file wasn't > contributed by Dremio, its header should be the default ASF one. > > Thanks, > Alex > > [4823]: https://github.com/apache/polaris/pull/4823 > > On Tue, Sep 22, 2026 at 10:58 AM Jean-Baptiste Onofré <[email protected]> > wrote: > > > > Hi everyone, > > > > I propose that we release the following RC as the official Apache Polaris > > 1.8.0 release. > > > > This corresponds to the tag: apache-polaris-1.8.0-rc1 > > > > * https://github.com/apache/polaris/commits/apache-polaris-1.8.0-rc1 > > * > > > https://github.com/apache/polaris/tree/69561122b8a1115b6ab1a2988eba7486fcfa2ab3 > > > > The release tarball, signature, and checksums are here: > > > > * https://dist.apache.org/repos/dist/dev/polaris/1.8.0 > > > > Helm charts are available on: > > > > * https://dist.apache.org/repos/dist/dev/polaris/helm-chart/1.8.0 > > > > NB: you have to build the Docker images locally in order to test Helm > > charts. > > > > The Python CLI wheel is available on: > > > > * https://dist.apache.org/repos/dist/dev/polaris/python-client/1.8.0 > > > > The Python CLI is also available on Test PyPI: > > > > * https://test.pypi.org/project/apache-polaris/1.8.0rc1/ > > > > You can find the KEYS file here: > > > > * https://downloads.apache.org/polaris/KEYS > > > > Convenience binary artifacts are staged on Nexus. The Maven repository > URL > > is: > > > > * > > > https://repository.apache.org/content/repositories/orgapachepolaris-1076/ > > > > Please download, verify, and test according to the release verification > > guide, which can be found at: > > > > * > > > https://polaris.apache.org/community/release-guides/release-verification-guide/ > > > > Please vote in the next 72 hours. > > > > [ ] +1 Release this as Apache Polaris 1.8.0 > > [ ] +0 > > [ ] -1 Do not release this because... > > > > Only PMC members have binding votes, but other community members are > > encouraged to cast non-binding votes. > > This vote will pass if there are 3 binding +1 votes and more binding +1 > > votes than -1 votes. > > > > Regards > > JB >
