lidavidm commented on a change in pull request #11111:
URL: https://github.com/apache/arrow/pull/11111#discussion_r707653875
##########
File path: cpp/src/arrow/compute/kernels/hash_aggregate.cc
##########
@@ -1737,6 +1737,12 @@ struct AntiExtrema {
static constexpr CType anti_max() { return
std::numeric_limits<CType>::min(); }
};
+template <>
+struct AntiExtrema<bool> {
+ static constexpr float anti_min() { return true; }
+ static constexpr float anti_max() { return false; }
Review comment:
Because I don't pay attention when I copy-paste, probably…fixed.
--
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
