From d4ff3e7d5ae6db76b64661e933e98ff6456d44c1 Mon Sep 17 00:00:00 2001
From: Ruoxi Sun <zanmato1984@gmail.com>
Date: Thu, 26 Sep 2024 17:26:04 +0800
Subject: [PATCH] Fix

---
 cpp/src/arrow/compute/row/grouper.cc | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/cpp/src/arrow/compute/row/grouper.cc b/cpp/src/arrow/compute/row/grouper.cc
index 73d326b427108..4bffb4ba08ba4 100644
--- a/cpp/src/arrow/compute/row/grouper.cc
+++ b/cpp/src/arrow/compute/row/grouper.cc
@@ -208,7 +208,7 @@ struct SimpleKeySegmenter : public BaseRowSegmenter {
     }
     ARROW_DCHECK(value.is_array());
     const auto& array = value.array;
-    DCHECK_GT(array.GetNullCount(), 0);
+    DCHECK_EQ(array.GetNullCount(), 0);
     return GetNextSegmentDeprecated(*array.type, GetValuesAsBytes(array), offset,
                                     batch.length);
     ARROW_UNSUPPRESS_DEPRECATION_WARNING
@@ -346,7 +346,7 @@ struct AnyKeysSegmenter : public BaseRowSegmenter {
     // `data` is an array whose index-0 corresponds to index `offset` of `batch`
     const std::shared_ptr<ArrayData>& data = datum.array();
     DCHECK_EQ(data->length, batch.length - offset);
-    ARROW_DCHECK(data->GetNullCount() == 0);
+    DCHECK_EQ(data->GetNullCount(), 0);
     DCHECK_EQ(data->type->id(), GroupIdType::type_id);
     const group_id_t* values = data->GetValues<group_id_t>(1);
     int64_t cursor;
@@ -419,7 +419,7 @@ struct AnyKeysSegmenter : public BaseRowSegmenter {
                                                         /*length=*/1));
     DCHECK(datum.is_array());
     const std::shared_ptr<ArrayData>& data = datum.array();
-    ARROW_DCHECK(data->GetNullCount() == 0);
+    DCHECK_EQ(data->GetNullCount(), 0);
     DCHECK_EQ(data->type->id(), GroupIdType::type_id);
     DCHECK_EQ(1, data->length);
     const group_id_t* values = data->GetValues<group_id_t>(1);
@@ -650,7 +650,7 @@ struct GrouperFastImpl : public Grouper {
       const TypeHolder& key = keys[icol];
       if (key.id() == Type::DICTIONARY) {
         auto bit_width = checked_cast<const FixedWidthType&>(*key).bit_width();
-        ARROW_DCHECK(bit_width % 8 == 0);
+        DCHECK(bit_width % 8, 0);
         impl->col_metadata_[icol] = KeyColumnMetadata(true, bit_width / 8);
       } else if (key.id() == Type::BOOL) {
         impl->col_metadata_[icol] = KeyColumnMetadata(true, 0);