[GitHub] [arrow-datafusion] Jimexist opened a new issue #1598: Follow up on the removal of `update` and `merge` in `Accumulator`

[GitBox <gi...@apache.org>]

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


   I think it would be a good idea as a follow on PR to update these implementations to avoid the use of ScalarValue. 
   
   I'll try find some time this week to do it (I need some coding time :) -- lots of reviewing so far)
   
   _Originally posted by @alamb in https://github.com/apache/arrow-datafusion/pull/1582#discussion_r786044972_


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

[GitHub] [arrow-datafusion] alamb commented on issue #1598: Follow up on the removal of `update` and `merge` in `Accumulator`

[GitBox <gi...@apache.org>]

alamb commented on issue #1598:
URL: https://github.com/apache/arrow-datafusion/issues/1598#issuecomment-1018746021


   Basically the idea here would be to rewrite the implementations that use `ScalarValue` to directly use the appriate arrow compute kernels: https://docs.rs/arrow/7.0.0/arrow/compute/index.html


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