-1 (binding)

Here's what I checked:
- KEYS has been updated and gpg server too
- Source distribution
-- signature and checksum are good
-- DISCLAIMER is present and good
-- LICENSE and NOTICE look good
-- no binary file found in the source distribution
-- Header look good in file (ASF/Dremio headers)
- Binary distribution
-- NB: there's no problem to use polaris-bin as name as soon as the
version contain incubating
-- signature and checksum are good
-- DISCLAIMER is present and good
-- *BLOCKER* LICENSE is not correct because:
--- caffeine 3.2.1 is documented in the LICENSE file, but caffeine
3.2.0 is actually bundled in the server/admin lib folder
--- reactor-core 3.7.6 is documented in the LICENSE file, but
reactor-core 3.7.7 is actually bundled in the server/admin lib folder
--- micrometer-* 1.15.1 are documented in the LICENSE file, but
micrometer-* 1.14.6 is actually bundled in the server/admin lib folder
--- hamcrest 2.2 is present in the server lib folder but not
documented in the LICENSE file
--- awaitility 4.3.0 is present in the server lib folder but not
documented in the LICENSE file

Nit (not blocker):
- helm chart is not in the right folder (see
https://github.com/apache/polaris/blob/main/site/content/release-guide.md?plain=1#L166)
- in 0.10.x, I staged a Docker image for review (see
https://github.com/apache/polaris/pull/1915/files#diff-4550e2201644991527d084c7d544c0c8b9ce97d41e129cc2a627a85949ae030bR186).
It's not a blocker for the release, we can always push the Docker
image after the release vote. I can also stage a Docker image on the
tag.
- polaris spark client bundle artifact contains two LICENSE and NOTICE
files in the root folder. Luckily the last extracted are the good ones
but it's not great. We should exclude LICENSE/NOTICE coming from
shaded dependencies to keep only our LICENSE/NOTICE file.
- the email body/subject txt files can be removed from the dist repository

I will work with Yufei to fix the LICENSE/NOTICE content in the binary
distribution.

Regards
JB





On Tue, Jun 24, 2025 at 3:23 AM Yufei Gu <flyrain...@gmail.com> wrote:
>
> Hi everyone,
>
> I propose that we release the following RC as the official
> Apache Polaris 1.0.0-incubating release.
>
> This corresponds to the tag: apache-polaris-1.0.0-incubating-rc0
> *
> https://github.com/apache/polaris/commits/apache-polaris-1.0.0-incubating-rc0
> *
> https://github.com/apache/polaris/tree/3e3cd23dfdc261686f3935c8ea2c472b0bd3e8b8
>
> The release tarball, signature, and checksums are here, which includes both
> source code distribution and a single binary distribution:
> *
> https://dist.apache.org/repos/dist/dev/incubator/polaris/apache-polaris-1.0.0-incubating
>
> You can find the KEYS file here:
> * https://downloads.apache.org/incubator/polaris/KEYS
>
> Convenience binary artifacts are staged on Nexus. The Maven repository URL
> is:
> *
> https://repository.apache.org/content/repositories/orgapachepolaris-1020/org/apache/polaris/
>
> Please download, verify, and test.
>
> Please vote in the next 72 hours.
>
> [ ] +1 Release this as Apache polaris 1.0.0-incubating
> [ ] +0
> [ ] -1 Do not release this because...
>
> Only PPMC members and mentors 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.
>
> NB: if this vote passes, a new vote has to be started on the Incubator
> general mailing
> list.
>
> Thanks
> Yufei

Reply via email to