khwilson commented on PR #44184: URL: https://github.com/apache/arrow/pull/44184#issuecomment-2409115768
@mapleFU I believe this is done now. Some notes on the diff: * The hash aggregates had to be updated as well (missed them in the first pass) * I've also added in Decimal32/64 support the basic aggregates (sum, product, mean, min/max, index). However, there's quite a lot of missing support for these types still in compute (most notably in casts) * Docs are updated to reflect the change And a note that quite a few tests are failing for what appears to be the same reason as #41390. Happy to address them if you'd like. -- 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]
