[GitHub] [iceberg] rdsr edited a comment on issue #1152: Why do we need two avro record readers & writers ?

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

rdsr edited a comment on issue #1152:
URL: https://github.com/apache/iceberg/issues/1152#issuecomment-653619954


   I think there's some confusion in how we name things. Reader/writer funcs in IcebergGenerics for ORC and Parquet are called `OrcGeneric[Reader|Writer]` and `GenericParquet[Reader|Writer]`, whereas for Avro they are called `Data[Reader|Writer]` . On top of that `GenericAvro[Reader|Writer] classes are internal and not used by IcebergGenerics


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