[
https://issues.apache.org/jira/browse/ARROW-10325?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Antoine Pitrou resolved ARROW-10325.
------------------------------------
Fix Version/s: 3.0.0
Resolution: Fixed
Issue resolved by pull request 8523
[https://github.com/apache/arrow/pull/8523]
> [C++][Compute] Separate aggregate kernel registration
> -----------------------------------------------------
>
> Key: ARROW-10325
> URL: https://issues.apache.org/jira/browse/ARROW-10325
> Project: Apache Arrow
> Issue Type: Improvement
> Components: C++
> Reporter: Yibo Cai
> Assignee: Yibo Cai
> Priority: Trivial
> Labels: pull-request-available
> Fix For: 3.0.0
>
> Time Spent: 40m
> Remaining Estimate: 0h
>
> We have basic aggregate kernel 'count/mean/sum/min_max' implemented in one
> file (plus simd version) and more complicated 'mode' and 'variance/stddev'
> kernels implemented in separated files.
> 'mode' and 'variance/stddev' kernels are now registered together with basic
> kernels in aggregate_basic.cc. And there are 'mode' and 'variance/stddev'
> kernel related function definitions in aggregate_basic_internal.h.
> This is not good. They should be moved from basic kernel source to their own
> implementation files and registered separately.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
