From 530d59757226b4e06ddd66c463947c244dcc14a2 Mon Sep 17 00:00:00 2001 From: David Mollitor Date: Wed, 4 Aug 2021 22:30:52 -0400 Subject: [PATCH] PARQUET-2072: Do Not Determine Both Min/Max for Binary Stats --- .../parquet/column/statistics/BinaryStatistics.java | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/parquet-column/src/main/java/org/apache/parquet/column/statistics/BinaryStatistics.java b/parquet-column/src/main/java/org/apache/parquet/column/statistics/BinaryStatistics.java index 6cd5395c7f..7715c079c6 100644 --- a/parquet-column/src/main/java/org/apache/parquet/column/statistics/BinaryStatistics.java +++ b/parquet-column/src/main/java/org/apache/parquet/column/statistics/BinaryStatistics.java @@ -54,9 +54,13 @@ private BinaryStatistics(BinaryStatistics other) { @Override public void updateStats(Binary value) { if (!this.hasNonNullValue()) { - initializeStats(value, value); - } else { - updateStats(value, value); + min = value.copy(); + max = value.copy(); + this.markAsNotEmpty(); + } else if (comparator().compare(min, value) > 0) { + min = value.copy(); + } else if (comparator().compare(max, value) < 0) { + max = value.copy(); } }