GitHub user kevinjqliu edited a comment on the discussion: `PrimitiveLiteral` of of `Decimal`
https://github.com/apache/iceberg-rust/issues/669 is now merged so now `rust_decimal` is replaced with `fastnum` I like what Renjie suggested. we're using `PrimitiveLiteral` as `PyPrimitiveLiteral` in the python binding right now, https://github.com/apache/iceberg-rust/blob/6165cd94c61c4e25af8d2e742c8d1671c9a5d01c/bindings/python/src/data_file.rs#L26-L48 seems like `Datum` and `PyDatum` might be the right abstraction GitHub link: https://github.com/apache/iceberg-rust/discussions/2062#discussioncomment-15643193 ---- This is an automatically sent email for [email protected]. To unsubscribe, please send an email to: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
