+1 * Validated signature and checksum * Ran RAT checks * Ran tests that didn't require Docker in Java 17
As a follow up, I think that we should move any tests that require Docker to integrationTest rather than test. We should try not to rely on Docker containers in normal unit tests because containers can make the tests take a lot longer. Docker is great for running services that would otherwise be a mess of mocked interfaces, but I would prefer that we keep them separate as integration tests. On Mon, Feb 10, 2025 at 11:45 AM Anurag Mantripragada < anuragmantr...@gmail.com> wrote: > +1 > > I verified signature, checksums, license, built and ran tests locally. > > Thanks for taking care of the release, Amogh! > > Thanks, > Anurag > > On Sun, Feb 9, 2025 at 10:39 PM Amogh Jahagirdar <2am...@gmail.com> wrote: > >> Hi Everyone, >> >> I propose that we release the following RC as the official Apache Iceberg >> 1.8.0 release. >> >> The commit ID is c277c2014a1b37fe755cfe37f173b6465bb8cb73 >> * This corresponds to the tag: apache-iceberg-1.8.0-rc0 >> * https://github.com/apache/iceberg/commits/apache-iceberg-1.8.0-rc0 >> * >> https://github.com/apache/iceberg/tree/c277c2014a1b37fe755cfe37f173b6465bb8cb73 >> >> The release tarball, signature, and checksums are here: >> * https://dist.apache.org/repos/dist/dev/iceberg/apache-iceberg-1.8.0-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-1182/ >> >> Please download, verify, and test. >> >> Please vote in the next 72 hours. >> >> [ ] +1 Release this as Apache Iceberg 1.8.0 >> [ ] +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. >> >
