stevenzwu commented on code in PR #13425:
URL: https://github.com/apache/iceberg/pull/13425#discussion_r2191047299
##########
core/src/main/java/org/apache/iceberg/PartitionStatsHandler.java:
##########
@@ -89,17 +91,57 @@ private PartitionStatsHandler() {}
NestedField.optional(11, "last_updated_at", LongType.get());
public static final NestedField LAST_UPDATED_SNAPSHOT_ID =
NestedField.optional(12, "last_updated_snapshot_id", LongType.get());
+ // Using default value for v3 field to support v3 reader reading file
written by v2
+ public static final NestedField DV_COUNT =
+ NestedField.required("dv_count")
+ .withId(13)
+ .ofType(Types.IntegerType.get())
+ .withInitialDefault(Literal.of(0))
Review Comment:
awesome job for adding a unit test for this
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org
