westonpace commented on PR #14352: URL: https://github.com/apache/arrow/pull/14352#issuecomment-1338932139
> AFAIU, aggregating tables is a valid use case that should be supported. @westonpace, let me know your thoughts. Aggregation of tables should be done by using a table source node and an exec plan. Not by using the `GroupBy` function in `aggregate.h`. That was the intent behind the comment: ``` /// Internal use only: helpers for PyArrow and testing HashAggregateKernels. /// For public use see arrow::compute::Grouper or create an execution plan /// and use an aggregate node. ``` -- 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]
