Fixup incorrect usages of checkNotNull

Author: Andrew Duffy

parquet-column/src/main/java/org/apache/parquet/filter2/predicate/Operators.java

@@ -185,12 +185,12 @@ public static final class Eq<T extends Comparable<T>> extends ColumnFilterPredic
 
     // value can be null
     Eq(Column<T> column, T value) {
-      super(column, value);
+      super(column, checkNotNull(value, "value"));
     }
 
     // value can be null
     Eq(Column<T> column, T value, ByteSignedness signedness) {
-      super(column, value, signedness);
+      super(column, checkNotNull(value, "value"), signedness);
     }
 
     @Override
@@ -204,12 +204,12 @@ public static final class NotEq<T extends Comparable<T>> extends ColumnFilterPre
 
     // value can be null
     NotEq(Column<T> column, T value) {
-      super(column, value);
+      super(column, checkNotNull(value, "value"));
     }
 
     // value can be null
     NotEq(Column<T> column, T value, ByteSignedness signedness) {
-      super(column, value, signedness);
+      super(column, checkNotNull(value, "value"), signedness);
     }
 
 
@@ -224,7 +224,7 @@ public static final class Lt<T extends Comparable<T>> extends ColumnFilterPredic
 
     // value cannot be null
     Lt(Column<T> column, T value) {
-      super(column, value);
+      super(column, checkNotNull(value, "value"));
     }
 
     // value cannot be null

parquet-hadoop/src/main/java/org/apache/parquet/filter2/statisticslevel/StatisticsFilter.java

@@ -68,8 +68,8 @@ public class StatisticsFilter implements FilterPredicate.Visitor<Boolean> {
   private static final boolean BLOCK_CANNOT_MATCH = true;
 
   public static boolean canDrop(FilterPredicate pred, List<ColumnChunkMetaData> columns) {
-    checkNotNull(pred, "predicate cannot be null");
-    checkNotNull(columns, "columns cannot be null");
+    checkNotNull(pred, "pred");
+    checkNotNull(columns, "columns");
     return pred.accept(new StatisticsFilter(columns));
   }