linxingyuan1102 commented on issue #227: ORC column map fix URL: https://github.com/apache/incubator-iceberg/pull/227#issuecomment-530149305 > Thanks for the update, @linxingyuan1102. It would be great to have your help getting the spec updated. In the end, I think Presto should depend on Iceberg to convert an ORC file's schema to an Iceberg schema so that we can more easily guarantee compatibility, but in theory we should be able to re-implement the conversion to a schema with IDs. @rdblue Sorry I missed replying before. I agree that it's better to have Iceberg do the conversion for Presto. But Presto uses its own implementation of ORC reader/writer so I'm not sure how much work it will involve on the Presto side and if it's worth the efforts to do so. But let's update the spec first. We can see later if it's easy to let Presto use Iceberg's conversion method.
---------------------------------------------------------------- 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]
