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
>
>

Reply via email to