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

Reply via email to