I'm also -1 until we get #12095
<https://github.com/apache/iceberg/pull/12095/files> backported to 1.7.x.
We don't want to release artifacts when we know parts of the NOTICE(s) need
to be corrected.

Thanks,
Amogh Jahagirdar

On Tue, Jan 28, 2025 at 1:44 PM Jean-Baptiste Onofré <j...@nanthrax.net>
wrote:

> Sorry (again, it's a long day): I meant this PR:
> https://github.com/apache/iceberg/pull/12095
>
> On Tue, Jan 28, 2025 at 7:38 PM Jean-Baptiste Onofré <j...@nanthrax.net>
> wrote:
> >
> > Just to be clear: we are working on new changes on NOTICE/LICENSE that
> > would be great to include in 1.7.x.
> >
> > See https://github.com/apache/polaris/pull/903
> >
> > I think it would be great to have this for 1.8.0 but also for 1.7.2.
> > That's why I voted -1 :)
> >
> > Thoughts ?
> >
> > Regards
> > JB
> >
> > On Tue, Jan 28, 2025 at 7:16 PM Jean-Baptiste Onofré <j...@nanthrax.net>
> wrote:
> > >
> > > -1 (non binding)
> > >
> > > I actually did a change that we should "partially revert": I changed
> > > the NOTICE to use bundle name, which is not optimal.
> > >
> > > I propose to cancel this rc0 to create a rc1 fixing name in NOTICE.
> > >
> > > Sorry about that.
> > >
> > > Regards
> > > JB
> > >
> > > On Sun, Jan 26, 2025 at 9:41 PM Jean-Baptiste Onofré <j...@nanthrax.net>
> wrote:
> > > >
> > > > Hi everyone,
> > > >
> > > > I propose that we release the following RC as the official Apache
> > > > Iceberg 1.7.2 release.
> > > >
> > > > The commit ID is c2105b2634becf68b3fdabd0ee6fb0b6e93d4f0c
> > > > * This corresponds to the tag: apache-iceberg-1.7.2-rc0
> > > > * https://github.com/apache/iceberg/commits/apache-iceberg-1.7.2-rc0
> > > > *
> https://github.com/apache/iceberg/tree/c2105b2634becf68b3fdabd0ee6fb0b6e93d4f0c
> > > >
> > > > The release tarball, signature, and checksums are here:
> > > > *
> https://dist.apache.org/repos/dist/dev/iceberg/apache-iceberg-1.7.2-rc0
> > > >
> > > > You can find the KEYS file here:
> > > > * https://downloads.apache.org/iceberg/KEYS
> > > >
> > > > Convenience binary artifacts are staged on Nexus. The Maven
> repository URL is:
> > > > *
> https://repository.apache.org/content/repositories/orgapacheiceberg-1180/
> > > >
> > > > For the changes, checkout the Milestone:
> > > > https://github.com/apache/iceberg/milestone/52?closed=1
> > > >
> > > > Please download, verify, and test. Please vote in the next 72 hours.
> > > >
> > > > [ ] +1 Release this as Apache Iceberg 1.7.2
> > > > [ ] +0
> > > > [ ] -1 Do not release this because...
> > > >
> > > > Only PMC members 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.
> > > >
> > > > Thanks,
> > > > Regards
> > > > JB & Fokko
>

Reply via email to