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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
