amogh-jahagirdar commented on code in PR #17035:
URL: https://github.com/apache/iceberg/pull/17035#discussion_r3509840961
##########
core/src/test/java/org/apache/iceberg/TestTrackedFileStruct.java:
##########
@@ -229,7 +355,24 @@ void testCopyWithoutStats() {
@Test
void testCopyWithStatsFilters() {
- TrackedFileStruct file = createTrackedFileWithStats();
+ TrackedFileStruct file =
Review Comment:
While it's duplicative compared to using the helper, at this level of
abstraction we are going for testing explicit crud/copyh stuff, and so it's
helpful to see what exactly is being set rather than having to go into the
helper.
--
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]
