Fixing case when not has_min_max_

apache · Jun 8, 2023 · bc598dd · bc598dd
1 parent 8d52868
commit bc598dd
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/cpp/src/parquet/statistics.cc b/cpp/src/parquet/statistics.cc
@@ -568,11 +568,13 @@ class TypedStatisticsImpl : public TypedStatistics<DType> {
       this->has_null_count_ = false;
     }
     // Distinct count cannot be merged.
-    this->has_distinct_count_ = false;
+    if (has_distinct_count_) {
+      has_distinct_count_ = false;
+    }
+    // If !other.HasMinMax, might be all-nulls all nulls and nan,
+    // so, not clear `this->has_min_max_` here.
     if (other.HasMinMax()) {
       SetMinMax(other.min(), other.max());
-    } else {
-      this->has_min_max_ = false;
     }
   }