Hi Dongjoon, JB, added the following additional files. Thanks Dongjoon for sharing the point that the ASF INFRA team may or may not admit *.prov files for signatures and checksums. This is an extra step to make sure the release can pass. We don't have to do that once we verify with the INFRA team if *.prov is valid.
- polaris-1.0.0-incubating.tgz.asc <https://dist.apache.org/repos/dist/dev/incubator/polaris/helm-chart/1.0.0-incubating/polaris-1.0.0-incubating.tgz.asc> - polaris-1.0.0-incubating.tgz.sha512 <https://dist.apache.org/repos/dist/dev/incubator/polaris/helm-chart/1.0.0-incubating/polaris-1.0.0-incubating.tgz.sha512> Thanks Robert and Dmitri for raising the concern of the jar name. We can definitely improve on it. but it shouldn't be a blocker for the 1.0.0 release IMHO. I'm pretty sure there is a lot to improve, but at this moment, the most important thing is to deliver a stable release that is highly demanded. Then we can get feedback and prioritize the improvement items. Yufei On Sat, Jul 5, 2025 at 11:43 AM yun zou <yunzou.colost...@gmail.com> wrote: > 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 > > > >> > > > >> > > > > > >
