I believe the _helpers.tpl file is mentioned in LICENSE (
https://github.com/apache/polaris/blob/apache-polaris-0.10.0-beta-incubating-rc3/LICENSE#L317).
Am I missing something?

--

Pierre


On Wed, May 21, 2025 at 6:14 AM Jean-Baptiste Onofré <j...@nanthrax.net>
wrote:

> Thanks Ryan
>
> If the helm chart tarball name is not a blocker, the license there is.
>
> We have to fix that.
>
> I will cancel rc3 to fix helm chart license and notice.
>
> Thanks
> Regards
> JB
>
> Le mar. 20 mai 2025 à 22:10, Ryan Blue <rdb...@gmail.com> a écrit :
>
> > Mostly checked licenses. There were a couple of minor issues:
> > - The helm charts tarball doesn't include "polaris" in the name.
> > - The helm charts tarball includes at least one file (_helpers.tpl) with
> a
> > Dremio copyright header that isn't documented in the LICENSE file
> >
> >
> > On Tue, May 20, 2025 at 10:09 AM Jean-Baptiste Onofré <j...@nanthrax.net>
> > wrote:
> >
> > > Hi
> > >
> > > Yes: no shaded jar artifacts on Maven repository.
> > >
> > > On maven repository you will find the same tar.gz/zip as on dist. The
> > other
> > > jars are “regular” jar files.
> > >
> > > Regards
> > > JB
> > >
> > > Le mar. 20 mai 2025 à 17:19, Ryan Blue <rdb...@gmail.com> a écrit :
> > >
> > > > JB, can you please confirm that there should be no shaded binaries in
> > the
> > > > staged maven repository? Any context like this you can give us will
> > help
> > > > when reviewing the licenses for this release. For example, if the
> helm
> > > > chart tarball doesn't contain third-party code that would be helpful
> to
> > > > know.
> > > >
> > > > On Tue, May 20, 2025 at 12:48 AM Dennis Huo <huoi...@gmail.com>
> wrote:
> > > >
> > > > > +1 (binding)
> > > > >
> > > > > 1. Verified checksums
> > > > > 2. Verified build/test from source distribution
> > > > > 3. Verified server binary distribution
> > > > >
> > > > > On Mon, May 19, 2025 at 2:40 PM Yufei Gu <flyrain...@gmail.com>
> > wrote:
> > > > >
> > > > > > +1(binding)
> > > > > >
> > > > > > 1. Verified asc, sha512.
> > > > > > 2. Build and test passed.
> > > > > > 3. Verified both binary distributions, admin and server. They can
> > run
> > > > > > successfully.
> > > > > >
> > > > > > Yufei
> > > > > >
> > > > > >
> > > > > > On Mon, May 19, 2025 at 1:39 PM Adnan Hemani
> > > > > > <adnan.hem...@snowflake.com.invalid> wrote:
> > > > > >
> > > > > > > (Non-binding) +1
> > > > > > >
> > > > > > > On Mon, May 19, 2025 at 1:26 PM Jean-Baptiste Onofré <
> > > > j...@nanthrax.net>
> > > > > > > wrote:
> > > > > > >
> > > > > > > > Hi everyone
> > > > > > > >
> > > > > > > > I propose that we release the following RC (RC3) as the
> > official
> > > > > > > > Apache Polaris 0.10.0-beta-incubating release.
> > > > > > > >
> > > > > > > > * This corresponds to the tag:
> > > > > > apache-polaris-0.10.0-beta-incubating-rc3
> > > > > > > > *
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> https://github.com/apache/polaris/commits/apache-polaris-0.10.0-beta-incubating-rc3
> > > > > > > > *
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> https://github.com/apache/polaris/tree/f04c59a78b6ede3bb73bf9dc373620d8e6d95e25
> > > > > > > >
> > > > > > > > The release source distribution tarball, signature, and
> > checksum
> > > > are
> > > > > > > > staged here:
> > > > > > > > * tgz:
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> https://dist.apache.org/repos/dist/dev/incubator/polaris/apache-polaris-0.10.0-beta-incubating/apache-polaris-0.10.0-beta-incubating.tar.gz
> > > > > > > > * sha512:
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> https://dist.apache.org/repos/dist/dev/incubator/polaris/apache-polaris-0.10.0-beta-incubating/apache-polaris-0.10.0-beta-incubating.tar.gz.sha512
> > > > > > > > * gpg:
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> https://dist.apache.org/repos/dist/dev/incubator/polaris/apache-polaris-0.10.0-beta-incubating/apache-polaris-0.10.0-beta-incubating.tar.gz.asc
> > > > > > > >
> > > > > > > > You can find the KEYS file here:
> > > > > > > > * https://downloads.apache.org/incubator/polaris/KEYS
> > > > > > > >
> > > > > > > > For convenience, the following binary artifacts are
> available:
> > > > > > > > * Staging Maven repository
> > > > > > > > **
> > > > > > > >
> > > > > >
> > > >
> > https://repository.apache.org/content/repositories/orgapachepolaris-1018
> > > > > > > > * Staging binary distributions (server and admin tool)
> tarball,
> > > > > > > > signature, and checksum:
> > > > > > > > ** Server:
> > > > > > > > *** tgz:
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> https://dist.apache.org/repos/dist/dev/incubator/polaris/apache-polaris-0.10.0-beta-incubating/polaris-quarkus-server-0.10.0-beta-incubating.tgz
> > > > > > > > *** tgz sha512:
> > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> https://dist.apache.org/repos/dist/dev/incubator/polaris/apache-polaris-0.10.0-beta-incubating/polaris-quarkus-server-0.10.0-beta-incubating.tgz.sha512
> > > > > > > > *** tgz gpg:
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> https://dist.apache.org/repos/dist/dev/incubator/polaris/apache-polaris-0.10.0-beta-incubating/polaris-quarkus-server-0.10.0-beta-incubating.tgz.asc
> > > > > > > > *** zip:
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> https://dist.apache.org/repos/dist/dev/incubator/polaris/apache-polaris-0.10.0-beta-incubating/polaris-quarkus-server-0.10.0-beta-incubating.zip
> > > > > > > > *** zip sha512:
> > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> https://dist.apache.org/repos/dist/dev/incubator/polaris/apache-polaris-0.10.0-beta-incubating/polaris-quarkus-server-0.10.0-beta-incubating.zip.sha512
> > > > > > > > *** zip gpg:
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> https://dist.apache.org/repos/dist/dev/incubator/polaris/apache-polaris-0.10.0-beta-incubating/polaris-quarkus-server-0.10.0-beta-incubating.zip.asc
> > > > > > > > ** Admin Tool:
> > > > > > > > *** tgz:
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> https://dist.apache.org/repos/dist/dev/incubator/polaris/apache-polaris-0.10.0-beta-incubating/polaris-quarkus-admin-0.10.0-beta-incubating.tgz
> > > > > > > > *** tgz sha512:
> > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> https://dist.apache.org/repos/dist/dev/incubator/polaris/apache-polaris-0.10.0-beta-incubating/polaris-quarkus-admin-0.10.0-beta-incubating.tgz.sha512
> > > > > > > > *** tgz gpg:
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> https://dist.apache.org/repos/dist/dev/incubator/polaris/apache-polaris-0.10.0-beta-incubating/polaris-quarkus-admin-0.10.0-beta-incubating.tgz.asc
> > > > > > > > *** zip:
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> https://dist.apache.org/repos/dist/dev/incubator/polaris/apache-polaris-0.10.0-beta-incubating/polaris-quarkus-admin-0.10.0-beta-incubating.zip
> > > > > > > > *** zip sha512:
> > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> https://dist.apache.org/repos/dist/dev/incubator/polaris/apache-polaris-0.10.0-beta-incubating/polaris-quarkus-admin-0.10.0-beta-incubating.zip.sha512
> > > > > > > > *** zip gpg:
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> https://dist.apache.org/repos/dist/dev/incubator/polaris/apache-polaris-0.10.0-beta-incubating/polaris-quarkus-admin-0.10.0-beta-incubating.zip.asc
> > > > > > > > * Staging Docker images:
> > > > > > > > ** Server:
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> https://hub.docker.com/layers/apache/polaris/0.10.0-beta-incubating-rc3/images/sha256-aeaf3c85d603131f4025ac615abb8f41e2d5f9f9349a4c879b2f01ce3de41147
> > > > > > > > * Helm charts:
> > > > > > > > ** tgz:
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> https://dist.apache.org/repos/dist/dev/incubator/polaris/helm-chart/0.10.0-beta-incubating.tar.gz
> > > > > > > > ** tgz sha512:
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> https://dist.apache.org/repos/dist/dev/incubator/polaris/helm-chart/0.10.0-beta-incubating.tar.gz.sha512
> > > > > > > > ** tgz gpg:
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> https://dist.apache.org/repos/dist/dev/incubator/polaris/helm-chart/0.10.0-beta-incubating.tar.gz.asc
> > > > > > > >
> > > > > > > > Please download, verify, and test.
> > > > > > > >
> > > > > > > > Please vote in the next 72 hours.
> > > > > > > >
> > > > > > > > [ ] +1 Release this as Apache polaris 0.10.0-beta-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.
> > > > > > > >
> > > > > > > > Thanks
> > > > > > > > Regards
> > > > > > > > JB
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
>

Reply via email to