zanmato1984 commented on code in PR #43720:
URL: https://github.com/apache/arrow/pull/43720#discussion_r1722030301


##########
cpp/src/arrow/compute/kernels/aggregate_basic.cc:
##########
@@ -272,26 +286,29 @@ void AddCountDistinctKernels(ScalarAggregateFunction* 
func) {
 // Sum implementation
 
 template <typename ArrowType>
-struct SumImplDefault : public SumImpl<ArrowType, SimdLevel::NONE> {
+struct SumImplNative : public SumImpl<ArrowType, SimdLevel::NONE> {

Review Comment:
   `Plain` maybe? If this doesn't look convincing to you then I'll vote 
`Default`. 



-- 
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-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to