alamb opened a new pull request, #9910: URL: https://github.com/apache/arrow-rs/pull/9910
## Summary This PR is stacked on #9905. It changes `FixedSizeBinaryArray` to store `value_size: usize` internally and uses `usize` arithmetic for internal address calculations. ## Details - stores the fixed-width element size as `usize` instead of `i32` - rewrites internal position calculations to use `usize` - removes the old `validate_lengths` invariant that existed only to keep internal `i32` offset arithmetic in range - keeps the public `i32`-based API surface where required by `DataType::FixedSizeBinary`, using checked conversions at those boundaries ## Validation - `cargo test -p arrow-array test_constructors` - `cargo test -p arrow-array test_fixed_size_binary_array_from_iter` - `cargo test -p arrow-array test_fixed_size_binary_array_from_sparse_iter` - `cargo test -p arrow-array test_value_offset_panics_when_offset_exceeds_i32` - `cargo test -p arrow-array --no-run` - `cargo +stable fmt --all --check` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
