fallintoplace opened a new issue, #2475: URL: https://github.com/apache/iceberg-rust/issues/2475
### Problem `Literal::try_from_json` does not currently support JSON single-value parsing for `binary` or `fixed` primitive types. Both branches call `todo!()`, so valid JSON values can panic during parsing. Binary JSON serialization also formats each byte without zero padding. For example, bytes like `[0, 1, 15, 255]` serialize as `"01fff"`, which cannot be decoded unambiguously as byte-oriented hex. ### Expected behavior - `binary` JSON literals should parse from hex strings and serialize back to two-character-per-byte lowercase hex. - `fixed[N]` JSON literals should use the same hex encoding and validate that the decoded value contains exactly `N` bytes. - Invalid hex input should return a normal error instead of panicking. ### Impact This affects metadata/default/literal paths that rely on JSON single-value serialization for binary-like primitive values. Returning validation errors and preserving byte boundaries makes the behavior consistent with the Iceberg JSON representation. -- 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]
