JeonDaehong commented on PR #3432: URL: https://github.com/apache/iceberg-rust/pull/3432#issuecomment-6145823955
Thanks for picking this up! I ran the same setup on this branch with other source types. int/long/date/bool/binary now work, but `identity(decimal)`, `truncate(decimal)` and `identity(uuid)` still hit the `unreachable!()` in `Display for Datum` (`Int128`/`UInt128` with `Unknown` type). I missed these in #3381, sorry. uuid could reuse the `Uuid` formatting; decimal is trickier since `Unknown` loses the scale. Would a non-panicking fallback arm make sense here? -- 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]
