> > -- *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
JB, thanks for identifying these license issues, these were introduced by changes after our license PR. Let's fix them. - helm chart is not in the right folder (see > > https://github.com/apache/polaris/blob/main/site/content/release-guide.md?plain=1#L166 > ) This will be fixed in the next RC. - in 0.10.x, I staged a Docker image for review (see My plan is to publish docker images after 1.0 release, which is suggested by the guide "NB: the docker images (polaris-server and polaris-admin) will be published on DockerHub once release vote passes." With that, I will cancel this RC due to the license issue. Yufei On Tue, Jun 24, 2025 at 9:42 AM Jean-Baptiste Onofré <j...@nanthrax.net> wrote: > -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 >
