Hi All,

I'm fine with manual helm index updates after the RC is approved, but I
think it's very error-prone and complicates validating helm charts in RCs.

Since PRs [3515] and [4154] already provide a foundation for automated helm
index generation, it would be nice to use it.

[3515] https://github.com/apache/polaris/pull/3515

[4154] https://github.com/apache/polaris/pull/4154

Cheers,
Dmitri.

On Fri, Apr 10, 2026 at 6:36 AM Jean-Baptiste Onofré <[email protected]>
wrote:

> -1 (binding)
>
> I checked:
> - source distribution
> -- checksum and signature are good
> -- LICENSE and NOTICE are good
> -- ASF header is present in expected file
> -- No binary/compiled file in the source distribution
> - binary distribution
> -- checksum and signature are good
> -- LICENSE and NOTICE are good
> - spark bundle jar (polaris-spark-3.5_2.13-1.4.0-bundle.jar on Maven
> Staging Repository)
> -- checksum and signature are good
> -- LICENSE and NOTICE are not correct. LICENSE mentions dependencies not
> actually bundled in the bundle jar file:
> --- jspecify
> --- jakarta annotation
> --- jakarta validation
> --- jakarta servlet
> --- micrometer
> --- commons compress
> --- commons codec
> --- commons io
> --- commons lang
> --- slf4j
> --- j2objc
> This should be removed from the spark bundle jar LICENSE.
>
> Two notes:
> 1. The txt files in dist are not release blocker. They can be removed
> anytime from dist dev or dist release. I would suggest to remove it now to
> avoid confusion.
> 2. Helm chart repo can be updated as a follow up (like Docker images), so
> not a release blocker. The core artifacts for this release are OK. The
> index file is provided for convenience. On previous releases, helm repo
> index was provided for testing, but it was not the one we maintain (the one
> on dist release is "merged").
>
> I will work on a PR to fix the spark plugin bundle jar LICENSE.
>
> Regards
> JB
>
> On Mon, Apr 6, 2026 at 6:37 PM Adnan Hemani via dev <
> [email protected]>
> wrote:
>
> > Hi everyone,
> >
> > I propose that we release the following RC as the official Apache Polaris
> > 1.4.0 release. (Do not worry, there is no RC0 for this release :)
> >
> > This corresponds to the tag: apache-polaris-1.4.0-rc1
> > * https://github.com/apache/polaris/commits/apache-polaris-1.4.0-rc1
> > *
> >
> >
> https://github.com/apache/polaris/tree/70f7a0288412f80197fca71d5f635ca221eb6331
> >
> > The release tarball, signature, and checksums are here:
> > * https://dist.apache.org/repos/dist/dev/polaris/1.4.0
> >
> > Helm charts are available on:
> > * https://dist.apache.org/repos/dist/dev/polaris/helm-chart/1.4.0
> >
> > NB: you have to build the Docker images locally in order to test Helm
> > charts.
> >
> > You can find the KEYS file here:
> > * https://downloads.apache.org/polaris/KEYS
> >
> > Convenience binary artifacts are staged on Nexus. The Maven repositories
> > URLs are:
> > *
> >
> https://repository.apache.org/content/repositories/orgapachepolaris-1063/
> >
> > 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 96 hours (extra time given due to Iceberg
> Summit).
> >
> > [ ] +1 Release this as Apache Polaris 1.4.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.
> >
> > Best,
> > Adnan Hemani
> >
>

Reply via email to