Re: [VOTE] Release Apache Polaris 0.9.0-incubating

2024-11-18 Thread Jean-Baptiste Onofré
NB: gradlew is not a binary file, but I think it would be great to address .keep and *.typed. Regards JB On Tue, Nov 19, 2024 at 7:04 AM Jean-Baptiste Onofré wrote: > > -1 (binding) > > I found that the source distribution includes binaries, especially > gradlew, .keep, *.typed, it has to exclud

Re: [VOTE] Release Apache Polaris 0.9.0-incubating

2024-11-18 Thread Michael Collado
+1 On Sat, Nov 16, 2024 at 10:00 PM Jean-Baptiste Onofré 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/

Re: [VOTE] Release Apache Polaris 0.9.0-incubating

2024-11-18 Thread Jean-Baptiste Onofré
-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é wrote: > > Hi everyone, > > I pro

Re: Table maintenance in Polaris @ Thu, Nov 7, 2024 9:00am – 10:00am (GMT-08)

2024-11-18 Thread Jean-Baptiste Onofré
Hi Yufei Not sure we got consensus in all details but the overall picture is ok for me. Let’s continue the details definition in the PR. Thanks ! Regards JB Le jeu. 14 nov. 2024 à 02:39, Yufei Gu a écrit : > Hi everyone, > > > Thank you for joining the table maintenance discussion today! We m

Re: [VOTE] Release Apache Polaris 0.9.0-incubating

2024-11-18 Thread Jean-Baptiste Onofré
NB: rat is executed via gradle. The excludes are defined in the gradle configuration. You should use the same excludes if you run rat standalone. The files you are mentioning are actually part of the excludes. Regards JB Le lun. 18 nov. 2024 à 23:02, Kevin Liu a écrit : > +1 non-binding. > > Do

Re: [VOTE] Release Apache Polaris 0.9.0-incubating

2024-11-18 Thread Russell Spitzer
OK, in that case :) +1 I also built and ran all tests, checked checksums and worried about licensing issues On Mon, Nov 18, 2024 at 5:10 PM Kevin Liu wrote: > I don't think there's a license issue. Looks like there's already a RAT > check in the build steps. > > https://github.com/apache/polari

Re: [VOTE] Release Apache Polaris 0.9.0-incubating

2024-11-18 Thread Kevin Liu
I don't think there's a license issue. Looks like there's already a RAT check in the build steps. https://github.com/apache/polaris/blob/e46c6cbb61e69dcb12775fa262c09437f8ee8a28/build.gradle.kts#L59-L124 And the files listed above are excluded. Best, Kevin Liu On Mon, Nov 18, 2024 at 3:03 PM Rus

Re: [VOTE] Release Apache Polaris 0.9.0-incubating

2024-11-18 Thread Russell Spitzer
Shouldn't we fix the license issues before the release? Seems like an important and easy thing to do. I also think we should be excluding the "site" directory from the release? On Mon, Nov 18, 2024 at 4:56 PM Dmitri Bourlatchkov wrote: > +1 (non-binding) > > Validated sha512 and signature. > >

Re: [VOTE] Release Apache Polaris 0.9.0-incubating

2024-11-18 Thread Dmitri Bourlatchkov
+1 (non-binding) Validated sha512 and signature. Cheers, Dmitri. On Mon, Nov 18, 2024 at 3:16 PM Jean-Baptiste Onofré wrote: > Hi Dmitri > > I updated dist with the correct sha512. > > I noticed several gradle issues during the release (in signature, > maven publication, etc). I will create th

Re: [VOTE] Release Apache Polaris 0.9.0-incubating

2024-11-18 Thread Kevin Liu
+1 non-binding. Downloaded artifacts, verified signatures, checksums, and licenses (see below). Ran tests successfully. ``` curl https://dist.apache.org/repos/dist/release/incubator/polaris/KEYS -o KEYS gpg --import KEYS svn checkout https://dist.apache.org/repos/dist/dev/incubator/polaris/0.9.0

Re: Table maintenance in Polaris @ Thu, Nov 7, 2024 9:00am – 10:00am (GMT-08)

2024-11-18 Thread Yufei Gu
Hi Folks, First PR is out. Please take a look. https://github.com/apache/polaris/pull/457/files Yufei On Wed, Nov 13, 2024 at 5:39 PM Yufei Gu wrote: > Hi everyone, > > > Thank you for joining the table maintenance discussion today! We made > significant progress, and here are the key takeawa

Re: [VOTE] Release Apache Polaris 0.9.0-incubating

2024-11-18 Thread Jean-Baptiste Onofré
Hi Dmitri I updated dist with the correct sha512. I noticed several gradle issues during the release (in signature, maven publication, etc). I will create the issues/PRs to fix that with the release process documentation. Regards JB On Mon, Nov 18, 2024 at 3:03 PM Dmitri Bourlatchkov wrote: >

Re: [VOTE] Release Apache Polaris 0.9.0-incubating

2024-11-18 Thread Jean-Baptiste Onofré
Hi Dmitri Yes, it's normal: the KEYS file contains the keys of the release manager. As I'm the first release manager, it contains my key. Release managers will add their key in the KEYS file. Regards JB On Mon, Nov 18, 2024 at 2:53 PM Dmitri Bourlatchkov wrote: > > Hi JB, > > I can see only you

Re: [VOTE] Release Apache Polaris 0.9.0-incubating

2024-11-18 Thread Jean-Baptiste Onofré
Hi That’s cause of the gradle plugin. I will update the file on dist. Le lun. 18 nov. 2024 à 15:03, Dmitri Bourlatchkov a écrit : > I noticed that the apache-polaris-0.9.0-incubating.sha512 file contains the > right hash, but it's format does not match the output of `sha512sum > apache-polaris

Re: [VOTE] Release Apache Polaris 0.9.0-incubating

2024-11-18 Thread Dmitri Bourlatchkov
I noticed that the apache-polaris-0.9.0-incubating.sha512 file contains the right hash, but it's format does not match the output of `sha512sum apache-polaris-0.9.0-incubating.tar.gz` -- files names are not listed after hashes... As a result, sha512sum -c apache-polaris-0.9.0-incubating.sha512 ret

Re: [VOTE] Release Apache Polaris 0.9.0-incubating

2024-11-18 Thread Dmitri Bourlatchkov
Hi JB, I can see only your key (one) in the KEYS file... Is that normal? Thanks, Dmitri. On Sun, Nov 17, 2024 at 1:00 AM Jean-Baptiste Onofré wrote: > Hi everyone, > > I propose that we release the following RC as the official Apache > Polaris 0.9.0-incubating release. > > * This corresponds t