diff --git a/.palantir/revapi.yml b/.palantir/revapi.yml index bd02d8cd2140..0ec3043a69f8 100644 --- a/.palantir/revapi.yml +++ b/.palantir/revapi.yml @@ -866,6 +866,11 @@ acceptedBreaks: old: "method void org.apache.iceberg.encryption.Ciphers::()" new: "method void org.apache.iceberg.encryption.Ciphers::()" justification: "Static utility class - should not have public constructor" + "1.4.0": + org.apache.iceberg:iceberg-core: + - code: "java.field.serialVersionUIDChanged" + new: "field org.apache.iceberg.util.SerializableMap.serialVersionUID" + justification: "Serialization is not be used" apache-iceberg-0.14.0: org.apache.iceberg:iceberg-api: - code: "java.class.defaultSerializationChanged" diff --git a/api/src/main/java/org/apache/iceberg/Scan.java b/api/src/main/java/org/apache/iceberg/Scan.java index 9a0cf4b2d0a7..339bc75336ba 100644 --- a/api/src/main/java/org/apache/iceberg/Scan.java +++ b/api/src/main/java/org/apache/iceberg/Scan.java @@ -83,8 +83,7 @@ public interface Scan> { * *

Column stats include: value count, null value count, lower bounds, and upper bounds. * - * @param requestedColumns column names for which to keep the stats. If null then all - * column stats will be kept, when {@link #includeColumnStats()} is set. + * @param requestedColumns column names for which to keep the stats. * @return a new scan based on this that loads column stats for specific columns. */ default ThisT includeColumnStats(Collection requestedColumns) { diff --git a/core/src/main/java/org/apache/iceberg/V2Metadata.java b/core/src/main/java/org/apache/iceberg/V2Metadata.java index 4962001c9650..ba6bb4a31960 100644 --- a/core/src/main/java/org/apache/iceberg/V2Metadata.java +++ b/core/src/main/java/org/apache/iceberg/V2Metadata.java @@ -563,7 +563,7 @@ public F copy() { @Override public F copyWithStats(Set requestedColumnIds) { - return wrapped.copyWithStats(requestedColumnIds); + throw new UnsupportedOperationException("Cannot copy IndexedDataFile wrapper"); } @Override diff --git a/core/src/main/java/org/apache/iceberg/util/SerializableMap.java b/core/src/main/java/org/apache/iceberg/util/SerializableMap.java index 032a7eead7f3..4f1046cc7cf3 100644 --- a/core/src/main/java/org/apache/iceberg/util/SerializableMap.java +++ b/core/src/main/java/org/apache/iceberg/util/SerializableMap.java @@ -26,6 +26,7 @@ import org.apache.iceberg.relocated.com.google.common.collect.Maps; public class SerializableMap implements Map, Serializable { + private static final long serialVersionUID = 1L; private final Map copiedMap; private transient volatile Map immutableMap;