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 >
