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

Posted to github@arrow.apache.org by "icexelloss (via GitHub)" <gi...@apache.org> on 2023/03/02 22:08:58 UTC

[GitHub] [arrow] icexelloss commented on pull request #34311: GH-32884: [C++] Add ordered aggregation

icexelloss commented on PR #34311:
URL: https://github.com/apache/arrow/pull/34311#issuecomment-1452616880

   @rtpsw I have finished reviewing the aggregate node changes and half way through the grouper changes. In addition to the comments I made here, I've also consolidated some the my comments into this PR to you branch:
   https://github.com/apache/arrow/pull/34311
   Please take a look and merge it if you can.
   
   So far the general approach looks good I will try to finish up reviewing grouper changes tomorrow.


-- 
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