rdblue commented on code in PR #18108: URL: https://github.com/apache/iceberg/pull/18108#discussion_r4149028149
########## core/src/main/java/org/apache/iceberg/TrackedFileStruct.java: ########## @@ -80,14 +82,16 @@ class TrackedFileStruct extends SupportsIndexProjection implements TrackedFile, private long[] splitOffsets = null; private int[] equalityIds = null; + private transient StructProjection partitionProjection = null; Review Comment: I think this is okay. The best option is to copy the partition data during serialization. That's what is actually tested, and it's a good solution. Since the reader is going to copy when it returns records, this should be safe. I'm going to resolve 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: [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]
