rdblue opened a new pull request, #6506:
URL: https://github.com/apache/iceberg/pull/6506

   This refactors the Avro read path to use the newly introduced 
`SchemaWithPartnerVisitor`. The purpose of this is to make the resolver a 
little more standard and make it easy to inject types that implement 
`StructProtocol` for the readers to use. Since the readers are constructed in 
several places (that all create a new `ConstructReader`) this is refactor is 
needed.
   
   I didn't quite get finished with this work. The next step is to remove 
`ConstructReader` and move all reader construction into `SchemaResolver`, then 
to pass a map of struct constructors into the resolver.


-- 
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