From e77270f790c2d2b6da108fbeefbfdd149925ac61 Mon Sep 17 00:00:00 2001 From: Sreesh Maheshwar Date: Sun, 7 Sep 2025 16:32:18 +0100 Subject: [PATCH] Nit: Move unchecked suppression down to violating assignment in `ParquetMetricsRowGroupFilter` --- .../apache/iceberg/parquet/ParquetMetricsRowGroupFilter.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/parquet/src/main/java/org/apache/iceberg/parquet/ParquetMetricsRowGroupFilter.java b/parquet/src/main/java/org/apache/iceberg/parquet/ParquetMetricsRowGroupFilter.java index 1ad346d39ab7..ed2398930efa 100644 --- a/parquet/src/main/java/org/apache/iceberg/parquet/ParquetMetricsRowGroupFilter.java +++ b/parquet/src/main/java/org/apache/iceberg/parquet/ParquetMetricsRowGroupFilter.java @@ -437,7 +437,6 @@ public Boolean notIn(BoundReference ref, Set literalSet) { } @Override - @SuppressWarnings("unchecked") public Boolean startsWith(BoundReference ref, Literal lit) { int id = ref.fieldId(); @@ -447,6 +446,7 @@ public Boolean startsWith(BoundReference ref, Literal lit) { return ROWS_CANNOT_MATCH; } + @SuppressWarnings("unchecked") Statistics colStats = (Statistics) stats.get(id); if (colStats != null && !colStats.isEmpty()) { if (allNulls(colStats, valueCount)) { @@ -486,7 +486,6 @@ public Boolean startsWith(BoundReference ref, Literal lit) { } @Override - @SuppressWarnings("unchecked") public Boolean notStartsWith(BoundReference ref, Literal lit) { int id = ref.fieldId(); Long valueCount = valueCounts.get(id); @@ -496,6 +495,7 @@ public Boolean notStartsWith(BoundReference ref, Literal lit) { return ROWS_MIGHT_MATCH; } + @SuppressWarnings("unchecked") Statistics colStats = (Statistics) stats.get(id); if (colStats != null && !colStats.isEmpty()) { if (mayContainNull(colStats)) {