Thanks everyone who participated in the vote for Release Apache Polaris 1.0.0 (rc6).
The vote result is: +1: 4 (binding), 6 (non-binding) +0: 0 (binding), 0 (non-binding) -1: 0 (binding), 0 (non-binding) A new vote is starting in the Apache Incubator general mailing list. Yufei On Sun, Jul 6, 2025 at 11:38 AM Yufei Gu <flyrain...@gmail.com> wrote: > 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 >> > > >> >> > > >> >> > > >> > >> >
