ravlio opened a new issue #418:
URL: https://github.com/apache/arrow-datafusion/issues/418


   Hello! I'm just learning Rust and maybe I'm wrong, please correct me, if so. 
I noticed that this part with key creation 
https://github.com/apache/arrow-datafusion/blob/174226c086a4838eab2a238853b4871c295c0189/datafusion/src/physical_plan/hash_aggregate.rs#L514
 is called on each row. And I see that you do match and downcast on each row on 
each grouping key. All this should add a lot of additional CPU instructions (I 
haven't tested it, just my thoughts). Isn't this a suboptimal approach? For 
instance, we could leverage generics or macro here. Thanks.


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

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


Reply via email to