nevi-me commented on pull request #8640: URL: https://github.com/apache/arrow/pull/8640#issuecomment-728739069
Hi @sweb, I'm providing general comments, I'll look at this in detail over the days. I see that you're using `i128` as the backing type. One of the reasons that's prevented us from making progress on this, is not having a good BigDecimal impl in Rust to use. The biggest hurdle will be converting `123.45f64` to decimals, as we'll need to do that for integration testing. On the JIRA itself, the plan was to add the type, then also: * add support on the IPC reader and writer * enable decimal types in integration teting I'm fine with us doing the above as follow-ups, as they'd be quite involved. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org