fallintoplace opened a new pull request, #2476: URL: https://github.com/apache/iceberg-rust/pull/2476
### Summary - implement JSON hex parsing for `binary` and `fixed` literals - serialize binary-like literals as two-character-per-byte lowercase hex - validate `fixed[N]` JSON values have exactly `N` decoded bytes - add coverage for valid hex, uppercase input, invalid hex, and fixed-size mismatches ### Why The previous JSON literal parser used `todo!()` for `binary` and `fixed`, so valid JSON values could panic. Serialization also emitted bytes without zero padding, which could collapse byte boundaries for values containing `0x00` through `0x0f`. Fixes #2475. ### Tests - `cargo fmt --check` - `cargo test -p iceberg json` - `cargo test -p iceberg spec::values` - `cargo test -p iceberg` -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
