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: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
