[ARROW-10325] [C++][Compute] Separate aggregate kernel registration

• Type: Improvement
• Status: Resolved
• Priority: Trivial
• Resolution: Fixed
• Fix Version/s: 3.0.0
• Component/s: C++
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/26315
• Assignee: Yibo Cai
• Reporter: Yibo Cai
• Created: 16/Oct/20 10:13
• Updated: 11/Jan/23 08:12
• Resolved: 09/Nov/20 16:00

Description

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.

Issue Links

• links to: GitHub Pull Request #8523