Agree with Russel. PR 1991 is intended to affect only the bundle JAR usage when using the --jars option. The --packages use case will continue to work as it does today. Since the JAR name already changes with each version update, I don’t believe the JAR name change alone should block the 1.0 release.
Best Regards, Yun On Fri, Jul 4, 2025 at 7:49 PM Russell Spitzer <russell.spit...@gmail.com> wrote: > 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 > > >> > > >> > > >
