return null instead of empty list

apache · Oct 14, 2023 · 22ed9e8 · 22ed9e8
1 parent 11cd94f
commit 22ed9e8
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/core/src/main/java/org/apache/iceberg/BaseFile.java b/core/src/main/java/org/apache/iceberg/BaseFile.java
@@ -30,7 +30,6 @@
 import org.apache.avro.specific.SpecificData;
 import org.apache.iceberg.avro.AvroSchemaUtil;
 import org.apache.iceberg.relocated.com.google.common.base.MoreObjects;
-import org.apache.iceberg.relocated.com.google.common.collect.ImmutableList;
 import org.apache.iceberg.relocated.com.google.common.collect.Lists;
 import org.apache.iceberg.relocated.com.google.common.primitives.Ints;
 import org.apache.iceberg.relocated.com.google.common.primitives.Longs;
@@ -463,7 +462,7 @@ public ByteBuffer keyMetadata() {
 
   @Override
   public List<Long> splitOffsets() {
-    return splitOffsets == null ? ImmutableList.of() : Longs.asList(splitOffsets);
+    return splitOffsets == null ? null : Longs.asList(splitOffsets);
   }
 
   long[] splitOffsetArray() {
@@ -472,7 +471,7 @@ long[] splitOffsetArray() {
 
   @Override
   public List<Integer> equalityFieldIds() {
-    return equalityIds == null ? ImmutableList.of() : Ints.asList(equalityIds);
+    return equalityIds == null ? null : Ints.asList(equalityIds);
   }
 
   @Override