Dandandan edited a comment on issue #790:
URL: 
https://github.com/apache/arrow-datafusion/issues/790#issuecomment-894658089


   Some profiling results reveal that a large part of the time now is spent in 
`ScalarValue::iter_to_array` (and related calls inside the aggregation code and 
`eq_array`.
   
   The first one could be partly resolved by updating/introducing a method that 
takes `&ScalarValue` values instead of an owned one which requires some more 
clones. That will require a lot of updating though probably, around the 
accumulators, etc.
   
   And it can get away when the actual array contents could be stored in an 
array directly from the start, which I think is the more longer term plan.


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


Reply via email to