Hi Shawn,

Thanks for working on this release.

+1 binding

- Verified checksums and signatures 
- Built from source and successfully ran tests
- Checked LICENSE and NOTICE
- No unexpected binaries were found
- Verified the download links are correct.

Just noting that I observed a similar flakiness with the test Kurtis mentioned, 
though I do not consider it a blocker for this release.

Sung

On 2026/05/06 07:56:14 Maximilian Michels wrote:
> +1 (non-binding)
> 
> 1. Verified the archive checksum and signature
> 2. Extracted and inspected the source code for binaries
> 3. Verified license files / headers
> 
> -Max
> 
> PS: Still one more binding vote needed!
> 
> On Tue, May 5, 2026 at 7:18 PM Kevin Liu <[email protected]> wrote:
> >
> > Bump, this needs 1 more binding vote :)
> >
> > On Fri, May 1, 2026 at 4:28 PM Kurtis Wright <[email protected]> 
> > wrote:
> >>
> >> +1 (non-binding)
> >>
> >> Verified on two machines:
> >>
> >> AL2 x86_64
> >> AL2 aarch64
> >>
> >>
> >> Checks performed on both:
> >>
> >> Signature: `gpg --verify` OK against KEYS from 
> >> https://downloads.apache.org/iceberg/KEYS
> >> Checksum: sha512 OK
> >> LICENSE and NOTICE present at the tarball root
> >> Binary files found include Avro manifest-list fixtures and Puffin `.bin` 
> >> fixtures under `crates/iceberg/testdata/`. Non-blocking nit: 
> >> `.idea/vcs.xml` is included in the tarball. Will create an issue and send 
> >> PR to remove IDE files if this wasn't intentional.
> >> License headers: `license-eye header check` found 0 invalid
> >>    ran via `go install 
> >> github.com/apache/skywalking-eyes/cmd/license-eye@latest` received 
> >> following error: apache/skywalking-eyes:latest` container image fails with 
> >> `exec format error` on linux/arm64 — appears amd64-only
> >> Built from source: `make build` passed on both (1 flaky test see note 
> >> below)
> >> Unit + doc tests: `make unit-test` passed on both
> >>
> >> One test showed flakiness during `make unit-test`:
> >> `catalog::memory::catalog::tests::test_update_table` failed once with
> >> assertion failed: table.metadata().last_updated_ms() < 
> >> updated_table.metadata().last_updated_ms()
> >> at crates/iceberg/src/catalog/memory/catalog.rs:1880
> >> I have cut the following minor changes to fix it:
> >>
> >> PR - https://github.com/apache/iceberg-rust/pull/2395
> >> Issue - https://github.com/apache/iceberg-rust/issues/2393
> >>
> >> On Wed, Apr 29, 2026 at 9:12 PM Jean-Baptiste Onofré <[email protected]> 
> >> wrote:
> >>>
> >>> +1 (non binding)
> >>>
> >>> See my comment about binary files present in the source distribution.
> >>>
> >>> I verified:
> >>> - signature and checksum are good
> >>> - LICENSE and NOTICE are correct
> >>> - ASF header is present in all expected files
> >>> - There are three compiled binary test data files present in the source 
> >>> distribution (empty-puffin-uncompressed.bin, 
> >>> sample-metric-data-compressed-zstd.bin, 
> >>> sample-metric-data-uncompressed.bin). Source distribution must not 
> >>> contain any binary/compiled file. It would be great to download these 
> >>> files at build time instead of being part of the source distribution. 
> >>> Worth to consider in future release.
> >>>
> >>> Regards
> >>> JB
> >>>
> >>> On Tue, Apr 28, 2026 at 11:28 PM Shawn Chang <[email protected]> 
> >>> wrote:
> >>>>
> >>>> Hello Apache Iceberg Rust Community,
> >>>>
> >>>> This is a call for a vote to release Apache Iceberg Rust version 0.9.1.
> >>>>
> >>>> The tag to be voted on is: v0.9.1-rc.3
> >>>>
> >>>> The release candidate artifacts are available at:
> >>>> https://dist.apache.org/repos/dist/dev/iceberg/apache-iceberg-rust-0.9.1-rc3/
> >>>>
> >>>> The KEYS file for verifying signatures is available at:
> >>>> https://downloads.apache.org/iceberg/KEYS
> >>>>
> >>>> The Git tag for this release is:
> >>>> https://github.com/apache/iceberg-rust/releases/tag/v0.9.1-rc.3
> >>>>
> >>>> Please download, verify, and test the release candidate.
> >>>>
> >>>> This vote will be open for at least 72 hours and will remain open until 
> >>>> the required number of votes is reached.
> >>>>
> >>>> Please vote accordingly:
> >>>> [ ] +1 Approve
> >>>> [ ] +0 No opinion
> >>>> [ ] -1 Disapprove (please provide a reason)
> >>>>
> >>>> To learn more about Apache Iceberg, please visit:
> >>>> https://rust.iceberg.apache.org/
> >>>>
> >>>> Checklist for reference:
> >>>> [ ] Download links are valid
> >>>> [ ] Checksums and signatures are correct
> >>>> [ ] LICENSE and NOTICE files are present
> >>>> [ ] No unexpected binary files are included
> >>>> [ ] All source files have ASF headers
> >>>> [ ] The project builds successfully from source
> >>>>
> >>>> For more details, please refer to:
> >>>> https://rust.iceberg.apache.org/release.html#how-to-verify-a-release
> >>>>
> >>>> Thanks,
> >>>> Shawn
> >>
> >>
> >>
> >> --
> >> Thank You and Cheers,
> >> Kurtis C. Wright
> 

Reply via email to