chenjunjiedada commented on issue #1152:
URL: https://github.com/apache/iceberg/issues/1152#issuecomment-653702927
@openinx and I had a discussion yesterday. Basically, we think it needs to
refactor two Avro readers/writers into one base Avro reader/writer, and then
has two different versions on iceberg-core and iceberg-generic just like what
#1162 does. Also, the `DataReader/Writer` are in iceberg-core, I 'm thinking
whether we could move that into iceberg-data module.
Another thing is to build a reader uses the generics data model so as to
replace `IndexedRecord` interface of `Datafile` and `ManifestFile` with
`Record` as you describe above.
For naming, so do we want to use the pattern `{target file format} + {data
model} + reader/writer`?
----------------------------------------------------------------
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:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
