Yibo Cai created ARROW-10325:
--------------------------------
Summary: [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
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)
