[GitHub] [iceberg] chenjunjiedada commented on pull request #1186: refactor avro reader and writer

[GitBox <gi...@apache.org>]

chenjunjiedada commented on pull request #1186:
URL: https://github.com/apache/iceberg/pull/1186#issuecomment-658152290


   @rdblue, just updated PR description. The first step for refactoring I 'd like to take is to move public Avro reader and writer to iceberg-data subproject so that users can avoid referring the reader/writer in iceberg-core, and then remove `DataReader` and `DataWriter` in one of the later release. It does look confusing between `GenericAvroReader` and `AvroGenericReader`, how about we rename the internal one to `AvroReader`?


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org