You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by "westonpace (via GitHub)" <gi...@apache.org> on 2023/05/08 16:58:21 UTC

[GitHub] [arrow] westonpace commented on pull request #34769: GH-33976: [Python] Remove usage of TableGroupBy helper in favor of pyarrow.acero.Declaration

westonpace commented on PR #34769:
URL: https://github.com/apache/arrow/pull/34769#issuecomment-1538725595

   @jorisvandenbossche 
   
   If you don't mind, keeping the standalone groupby_test would be helpful.  I want to encourage splitting plan_test up into tests-per-node (see order_by_node_test and fetch_node_test which are similar in size to groupby_test).  Maybe you could rename it to aggregate_node_test?


-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org