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

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2021/06/10 09:12:16 UTC

[GitHub] [arrow] Crystrix commented on pull request #10486: ARROW-13016: [C++][Compute] Support Null type in Sum/Mean/MinMax aggregation

Crystrix commented on pull request #10486:
URL: https://github.com/apache/arrow/pull/10486#issuecomment-858454595


   Maybe we can keep a consistent behavior with numeric types. 
   For example,
   - Count: keep the current implementation (return 0 for `skip_nulls` option, otherwise return the count of the null elements)
   - Sum: return 0 if `min_count` == 0, otherwise return a null scalar
   - Mean: return a double scalar of null value
   - MinMax: return a struct scalar of two null scalar children


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

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