+1

It’s great to see the progress!

I did find a couple of minor things:

   - RAT checks exclude all **.md files, which I don’t think is correct.
   Luckily, commenting out that line and running RAT shows that all of the
   markdown files have good license headers. Because the markdown has headers,
   I don’t think this is a blocker.
   - The NOTICE file includes the entire Netty notice via the Nessie
   NOTICE, but the only content from Nessie are build files so this almost
   certainly doesn’t apply. It would be nice to clarify this and remove it so
   that downstream consumers don’t need to worry about it.

Other checks:

   - Validated signature and checksum
   - Ran RAT check and validated exclusions
   - Checked LICENSE and NOTICE changes


On Thu, Feb 6, 2025 at 8:55 AM Jean-Baptiste Onofré <j...@nanthrax.net> wrote:

> Hi folks,
>
> We did new fixes on NOTICE and LICENSE fix in the source distribution.
>
> This is the vote for Apache Polaris 0.9.0-incubating rc5.
>
> * This corresponds to the tag: apache-polaris-0.9.0-incubating-rc5
> *
> https://github.com/apache/polaris/tree/apache-polaris-0.9.0-incubating-rc5
> *
> https://github.com/apache/polaris/tree/829dff6ec4728bf70e8a772c42bccb519679574e
>
> The release source 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
>
> NB: this release doesn't provide any binary distribution, it provides
> source distribution only.
>
> 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
>

Reply via email to