edgarRd opened a new pull request #3312:
URL: https://github.com/apache/iceberg/pull/3312


   Adding name mapping to readers built in `IcebergInputFormat`. This will 
allow Hive to read data imported during Hive -> Iceberg table migration, which 
otherwise fails with NPE while trying to resolve the Iceberg types in the 
parquet/orc files.
   
   Since the change is pretty straightforward - and we have this same behavior 
in all other readers - I've not added more unit tests. We also already have 
extensive name mapping unit tests. However, if folks feel we need a unit test 
for this I'm happy to add it. I've validated this change by reading via Hive 
using the `IcebergInputFormat` on a migrated table.
   
   PTAL @pvary @aokolnychyi 


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

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to