Steven's comment

Author: pvary

parquet/src/main/java/org/apache/iceberg/parquet/ParquetFormatModel.java

@@ -222,17 +222,17 @@ public FileAppender<D> build() throws IOException {
   private static class ReadBuilderWrapper<R, D, S> implements ReadBuilder<D, S> {
     private final Parquet.ReadBuilder internal;
     private final ReaderFunction<R, S, MessageType> readerFunction;
-    private final boolean batchReader;
+    private final boolean isBatchReader;
     private S engineSchema;
     private Map<Integer, ?> idToConstant = ImmutableMap.of();
 
     private ReadBuilderWrapper(
         InputFile inputFile,
         ReaderFunction<R, S, MessageType> readerFunction,
-        boolean batchReader) {
+        boolean isBatchReader) {
       this.internal = Parquet.read(inputFile);
       this.readerFunction = readerFunction;
-      this.batchReader = batchReader;
+      this.isBatchReader = isBatchReader;
     }
 
     @Override
@@ -297,21 +297,21 @@ public ReadBuilder<D, S> withNameMapping(NameMapping nameMapping) {
 
     @Override
     public CloseableIterable<D> build() {
-      return batchReader
-          ? internal
-              .createBatchedReaderFunc(
-                  (icebergSchema, messageType) ->
-                      (VectorizedReader<?>)
-                          readerFunction.read(
-                              icebergSchema, messageType, engineSchema, idToConstant))
-              .build()
-          : internal
-              .createReaderFunc(
-                  (icebergSchema, messageType) ->
-                      (ParquetValueReader<?>)
-                          readerFunction.read(
-                              icebergSchema, messageType, engineSchema, idToConstant))
-              .build();
+      if (isBatchReader) {
+        return internal
+            .createBatchedReaderFunc(
+                (icebergSchema, messageType) ->
+                    (VectorizedReader<?>)
+                        readerFunction.read(icebergSchema, messageType, engineSchema, idToConstant))
+            .build();
+      } else {
+        return internal
+            .createReaderFunc(
+                (icebergSchema, messageType) ->
+                    (ParquetValueReader<?>)
+                        readerFunction.read(icebergSchema, messageType, engineSchema, idToConstant))
+            .build();
+      }
     }
   }
 }