Hi Yufei (as a release manager for 1.0.0): Even though RC6 has approving votes so far, let's resolve this concern (quoted below) before closing the vote.
Thanks, Dmitri. On Fri, Jul 4, 2025 at 9:48 AM Dmitri Bourlatchkov <di...@apache.org> wrote: > Good point, Robert! I missed the artifact name change while reviewing > PR 1991. > > With RC6 this Spark option worked for me (no classifier): > > --packages org.apache.polaris:polaris-spark-3.5_2.12:1.0.0-incubating > > In RC6 the bundle jar (polaris-spark-3.5_2.12-1.0.0-incubating-bundle.jar) > is under the same maven artifact name (with classifier). I believe it is > this jar that is going to be renamed in PR 1991. > > I do not think the bundle jar was meant for download via Maven. IIRC, it > was meant for manual use with the --jars Spark option. > > Nonetheless, I think it would be nicer from the user's perspective to > settle jar names before 1.0.0. I'd welcome finishing #1991 and including it > into 1.0.0 (new RC). > > Cheers, > Dmitri. > > On Fri, Jul 4, 2025 at 7:27 AM Robert Stupp <sn...@snazy.de> wrote: > >> I haven't checked the release yet due to other things on my plate. >> >> With PR #1991 (Add polaris-spark-bundle project to help packing and >> publish the Spark Client shadow Jar) [1], the Maven coordinates of the >> Spark plugin will change pretty much immediately after 1.0.0, which is >> relevant when using Spark's `--packages` option. >> >> In my vote for RC0, I objected "IIUC having a Maven classifier for the >> Spark plugin was considered an issue, however the "end user" dependency >> requires a Maven classifier (`bundle`). ", which is also relevant when >> using Spark's `--packages` option. >> >> Thoughts? >> >> >> [1] https://github.com/apache/polaris/pull/1991 >> >> >> On 7/2/25 20:55, Yufei Gu 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-rc6 >> > * >> > >> https://github.com/apache/polaris/commits/apache-polaris-1.0.0-incubating-rc6 >> > * >> > >> https://github.com/apache/polaris/tree/a701f105c5d44565ac0ea86db45edbcebdbed718 >> > NB: it's exactly the same as RC5 except for this commit: >> > >> https://github.com/apache/polaris/commit/a701f105c5d44565ac0ea86db45edbcebdbed718 >> > >> > The release tarball, signature, and checksums are here, including both >> > source code and binary distributions: >> > * >> > >> https://dist.apache.org/repos/dist/dev/incubator/polaris/apache-polaris-1.0.0-incubating/ >> > >> > >> > A binary package for Helm chart: >> > >> > * >> > >> https://dist.apache.org/repos/dist/dev/incubator/polaris/helm-chart/1.0.0-incubating/ >> > NB: File >> > >> https://dist.apache.org/repos/dist/dev/incubator/polaris/helm-chart/1.0.0-incubating/polaris-1.0.0-incubating.tgz.prov >> > contains >> > both signature and checksum for the package. Please verify it with the >> > command `helm verify`. >> > The docker images (polaris-server and polaris-admin) will be published >> on >> > DockerHub once the release vote passes. >> > >> > 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-1027/ >> > >> > 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. >> > >> > Yufei >> > >> -- >> Robert Stupp >> @snazy >> >>
