rdblue commented on code in PR #9366: URL: https://github.com/apache/iceberg/pull/9366#discussion_r1435370012
########## core/src/main/java/org/apache/iceberg/avro/ValueReaders.java: ########## @@ -564,6 +678,115 @@ public Map<K, V> read(Decoder decoder, Object reuse) throws IOException { } } + public abstract static class PlannedStructReader<S> + implements ValueReader<S>, SupportsRowPosition { Review Comment: This is the new base class that implements a read plan, which is a list of positions and readers. The read plan is produced by the new logic in `GenericAvroReader` that handles structs. -- 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