rename new optional ValueStatistics::min to min_opt

parquet/src/arrow/arrow_reader/statistics.rs

@@ -131,7 +131,7 @@ macro_rules! make_stats_iterator {
 
 make_stats_iterator!(
     MinBooleanStatsIterator,
-    min_unchecked,
+    min,
     ParquetStatistics::Boolean,
     bool
 );
@@ -141,33 +141,13 @@ make_stats_iterator!(
     ParquetStatistics::Boolean,
     bool
 );
-make_stats_iterator!(
-    MinInt32StatsIterator,
-    min_unchecked,
-    ParquetStatistics::Int32,
-    i32
-);
+make_stats_iterator!(MinInt32StatsIterator, min, ParquetStatistics::Int32, i32);
 make_stats_iterator!(MaxInt32StatsIterator, max, ParquetStatistics::Int32, i32);
-make_stats_iterator!(
-    MinInt64StatsIterator,
-    min_unchecked,
-    ParquetStatistics::Int64,
-    i64
-);
+make_stats_iterator!(MinInt64StatsIterator, min, ParquetStatistics::Int64, i64);
 make_stats_iterator!(MaxInt64StatsIterator, max, ParquetStatistics::Int64, i64);
-make_stats_iterator!(
-    MinFloatStatsIterator,
-    min_unchecked,
-    ParquetStatistics::Float,
-    f32
-);
+make_stats_iterator!(MinFloatStatsIterator, min, ParquetStatistics::Float, f32);
 make_stats_iterator!(MaxFloatStatsIterator, max, ParquetStatistics::Float, f32);
-make_stats_iterator!(
-    MinDoubleStatsIterator,
-    min_unchecked,
-    ParquetStatistics::Double,
-    f64
-);
+make_stats_iterator!(MinDoubleStatsIterator, min, ParquetStatistics::Double, f64);
 make_stats_iterator!(MaxDoubleStatsIterator, max, ParquetStatistics::Double, f64);
 make_stats_iterator!(
     MinByteArrayStatsIterator,
@@ -264,7 +244,7 @@ macro_rules! make_decimal_stats_iterator {
 
 make_decimal_stats_iterator!(
     MinDecimal128StatsIterator,
-    min_unchecked,
+    min,
     min_bytes,
     i128,
     from_bytes_to_i128
@@ -278,7 +258,7 @@ make_decimal_stats_iterator!(
 );
 make_decimal_stats_iterator!(
     MinDecimal256StatsIterator,
-    min_unchecked,
+    min,
     min_bytes,
     i256,
     from_bytes_to_i256

parquet/src/arrow/arrow_writer/mod.rs

@@ -2544,7 +2544,7 @@ mod tests {
                 assert!(stats.has_min_max_set());
                 if let Statistics::Int32(stats) = stats {
                     assert_eq!(
-                        *stats.min().unwrap() as u32,
+                        *stats.min_opt().unwrap() as u32,
                         *src_slice.iter().min().unwrap()
                     );
                     assert_eq!(
@@ -2591,7 +2591,7 @@ mod tests {
                 assert!(stats.has_min_max_set());
                 if let Statistics::Int64(stats) = stats {
                     assert_eq!(
-                        *stats.min().unwrap() as u64,
+                        *stats.min_opt().unwrap() as u64,
                         *src_slice.iter().min().unwrap()
                     );
                     assert_eq!(
@@ -3081,7 +3081,7 @@ mod tests {
 
         // Column chunk of column "a" should have chunk level statistics
         if let Statistics::ByteArray(byte_array_stats) = a_col.statistics().unwrap() {
-            let min = byte_array_stats.min().unwrap();
+            let min = byte_array_stats.min_opt().unwrap();
             let max = byte_array_stats.max_opt().unwrap();
 
             assert_eq!(min.as_bytes(), &[b'a']);
@@ -3153,7 +3153,7 @@ mod tests {
 
         // Column chunk of column "a" should have chunk level statistics
         if let Statistics::ByteArray(byte_array_stats) = a_col.statistics().unwrap() {
-            let min = byte_array_stats.min().unwrap();
+            let min = byte_array_stats.min_opt().unwrap();
             let max = byte_array_stats.max_opt().unwrap();
 
             assert_eq!(min.as_bytes(), &[b'a']);