[GitHub] [incubator-iceberg] rdsr edited a comment on issue #170: Add support for Iceberg MR / InputFormat and OutputFormat APIs

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

rdsr edited a comment on issue #170: Add support for Iceberg MR / InputFormat and OutputFormat APIs
URL: https://github.com/apache/incubator-iceberg/issues/170#issuecomment-519263693
 
 
   All good points.
   
   > I'd use Iceberg's Record interface instead. GenericRecord is the implementation class. It's also the name of an Avro class, which we shouldn't expose
   
   For this I think it should be possible to parameterize [using generics] the container and provide a default implementation for Iceberg's `Record` type.   Users may want to directly read Avro data or in their own container, they shouldn't have to re-convert from Iceberg's `Record` to, say, Avro. 

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


With regards,
Apache Git Services

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