update impl HeapSize for ValueStatistics to use new min and max api

apache · Aug 9, 2024 · 6950994 · 6950994
1 parent 87c375a
commit 6950994
Showing 1 changed file with 1 addition and 8 deletions.
diff --git a/parquet/src/file/metadata/memory.rs b/parquet/src/file/metadata/memory.rs
@@ -183,14 +183,7 @@ impl<T: ParquetValueType> HeapSize for PageIndex<T> {
 
 impl<T: ParquetValueType> HeapSize for ValueStatistics<T> {
     fn heap_size(&self) -> usize {
-        if self.has_min_max_set() {
-            return self.min_unchecked().heap_size() + self.max_unchecked().heap_size();
-        } else if self.min_is_exact() {
-            return self.min_unchecked().heap_size();
-        } else if self.max_is_exact() {
-            return self.max_unchecked().heap_size();
-        }
-        0
+        self.min().map(T::heap_size).unwrap_or(0) + self.max().map(T::heap_size).unwrap_or(0)
     }
 }
 impl HeapSize for bool {