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

Posted to github@arrow.apache.org by "mingmwang (via GitHub)" <gi...@apache.org> on 2023/07/05 13:28:56 UTC

[GitHub] [arrow-datafusion] mingmwang commented on issue #4973: Improve the performance of `Aggregator`, grouping, aggregation

mingmwang commented on issue #4973:
URL: https://github.com/apache/arrow-datafusion/issues/4973#issuecomment-1621762170

   > @mingmwang @Dandandan and @tustvold (and everyone else following along) here is a POC #6800
   > 
   > It follows similar pattern as you describe in [#4973 (comment)](https://github.com/apache/arrow-datafusion/issues/4973#issuecomment-1614211204) and It also makes q17 go more than 2x faster (and has no `unsafe` code or row format)!
   > 
   > > This branch: Query 17 avg time: 766.31 ms
   > > main: Query 17 avg time: 1789.73 ms
   > 
   > I would love feedback and thoughts on the approach
   
   I like your approach !!. The POC result is very promising, and the performance is better than my best version.


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