Thanks for flagging this Dmitri. In fact I think this could be a release blocker as the EclipseLink workflow is quite fundamentally broken. I commented on #422 <https://github.com/apache/polaris/pull/422> with some thoughts.
If we decide this is a release blocker, I would suggest we consider a revert of #438 <https://github.com/apache/polaris/pull/438>. I proposed this on #450 <https://github.com/apache/polaris/issues/450> but I believe @snazy had some concerns about that. --EM On Wed, Nov 20, 2024 at 12:22 PM Dmitri Bourlatchkov <dmitri.bourlatch...@dremio.com.invalid> wrote: > I've just noticed that RC1 includes > https://github.com/apache/polaris/pull/438, which, I think, makes > bootstrapping on EclispeLink effectively impossible, because there's no > user-level way to discover the generated root secret. > > From my personal point of view it is not a release blocker, so I'm keeping > my previous +1 vote, but I wanted to raise this concern in case it is > important for other people. > > Cheers, > Dmitri. > > > On Tue, Nov 19, 2024 at 11:37 AM Jean-Baptiste Onofré <j...@nanthrax.net> > wrote: > > > Actually, reverting my vote to +1 (binding) > > > > NB > > - .keep and .typed are actually empty files (even if flagged as > > binary), so not a problem > > - I will fix all gradle related issue during release process and > > documentation (already working on fix about digest sourceTarball and > > Maven publication) > > > > Regards > > JB > > > > On Tue, Nov 19, 2024 at 6:04 AM Jean-Baptiste Onofré <j...@nanthrax.net> > > wrote: > > > > > > -1 (binding) > > > > > > I found that the source distribution includes binaries, especially > > > gradlew, .keep, *.typed, it has to excluded from the source > > > distribution > > > > > > I will cancel this vote to fix that in rc2. > > > > > > Regards > > > JB > > > > > > On Sun, Nov 17, 2024 at 7:00 AM Jean-Baptiste Onofré <j...@nanthrax.net> > > wrote: > > > > > > > > Hi everyone, > > > > > > > > I propose that we release the following RC as the official Apache > > > > Polaris 0.9.0-incubating release. > > > > > > > > * This corresponds to the tag: apache-polaris-0.9.0-incubating-rc1 > > > > * > > > https://github.com/apache/polaris/commits/apache-polaris-0.9.0-incubating-rc1 > > > > * > > > https://github.com/apache/polaris/tree/445c42768d1e3148c912ac5c45ee53036b9ef318 > > > > > > > > The release tarball, signature, and checksums are here: > > > > * > > > https://dist.apache.org/repos/dist/dev/incubator/polaris/0.9.0-incubating/ > > > > > > > > You can find the KEYS file here: > > > > * https://dist.apache.org/repos/dist/release/incubator/polaris/KEYS > > > > > > > > Convenience binary artifacts are staged on Nexus. The Maven > repository > > URL is: > > > > * > > > https://repository.apache.org/content/repositories/orgapachepolaris-1003/ > > > > > > > > Please download, verify, and test. > > > > > > > > Please vote in the next 72 hours. > > > > [ ] +1 Release this as Apache polaris 0.9.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 will be started on the Incubator > > > > general mailing > > > > list. > > > > > > > > Thanks > > > > Regards > > > > JB > > >
