liamzwbao commented on issue #8059: URL: https://github.com/apache/arrow-rs/issues/8059#issuecomment-3166349362
Hi @alamb, IIUC, we should convert `Decimal32` → `VariantDecimal4`, `Decimal64` → `VariantDecimal8`, and `Decimal128` → `VariantDecimal16`. Do we already have a util function for this in the repo? If not, I can create one. Also, how should we handle `Decimal256`? Since there’s no `VariantDecimal32` equivalent, should we try casting it to `VariantDecimal16` and panic on overflow? Thank you very much! -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
