westonpace commented on PR #14352: URL: https://github.com/apache/arrow/pull/14352#issuecomment-1340597382
> IIUC, support of chunked arrays is needed only for GroupBy as a testing facility (and that this is https://github.com/apache/arrow/pull/14352/#issuecomment-1328279070 too), right? So, if we remove this support, some current testing using GroupBy would break and would need to be removed too. @westonpace, are you fine with this outcome? I have created https://github.com/apache/arrow/pull/14867 which removes the internal GroupBy facility entirely (well, remaps it onto exec plans). -- 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]
