Since all the artifacts are going to change their names when Polaris graduates (hopefully) I don’t think we should over index on having everything settled at this moment
Given that this is part of an experimental feature it doesn’t seem to me like it’s worth delaying the release for alignment on the jar name for the bundle artifact when the most common usage will say the same. Especially since if we graduate between now and the next release we would be changing it anyway. On Fri, Jul 4, 2025 at 11:37 AM Dmitri Bourlatchkov <di...@apache.org> wrote: > 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 > >> > >> >
