cbb330 commented on code in PR #16692:
URL: https://github.com/apache/iceberg/pull/16692#discussion_r3555084942
##########
data/src/test/java/org/apache/iceberg/data/TestMetricsRowGroupFilter.java:
##########
@@ -471,6 +472,86 @@ public void testColumnNotInFile() {
}
}
+ @TestTemplate
+ public void testColumnNotInFileWithInitialDefault() {
Review Comment:
Thanks for the suggestion, Steven. Added all three:
* startsWith/notStartsWith on country ("U" matches and "X" skips, with
symmetric notStartsWith cases).
* A DateType field using its raw Integer day-ordinal representation for
lt/gt/eq, guarding the unchecked cast.
* A DoubleType field with a non-NaN default covering isNaN (skip) and notNaN
(match).
All follow the existing “field ID absent from the file schema” pattern with
no new fixtures. Happy to add decimal or binary coverage if useful.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
