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

Posted to github@arrow.apache.org by "alamb (via GitHub)" <gi...@apache.org> on 2023/07/27 14:58:06 UTC

[GitHub] [arrow-datafusion] alamb commented on issue #7112: Remove AggregateExpr::create_accumulator

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

   I think updating `AggregateExpr` which is largely an internal implementation detail of DataFusion, to better communicate what is supported and what is not makes a lot of sense to me.
   
   I think there are two accumulators now because Accumulator has an API to do retractable updates (incremental updates)
   Accumulator is part of the public API so I don't think it can / should be removed


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