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

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/04/29 19:02:57 UTC

[GitHub] [arrow-datafusion] andygrove commented on pull request #2375: WIP: Use row format for aggregate

andygrove commented on PR #2375:
URL: https://github.com/apache/arrow-datafusion/pull/2375#issuecomment-1113633390

   > The current PR seems scary in size, maybe I should move the physical_plan folder re-org as a separate PR first.
   
   I think that would help.
   
   Are we replacing HashAggregate completely with a new row based aggregate or do we want to support both? Does hash aggregate still have advantages for some use cases? Maybe we can have a config setting for which one to use?


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