[GitHub] [incubator-iceberg] linxingyuan1102 commented on issue #227: ORC column map fix

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

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