pvary commented on pull request #3947:
URL: https://github.com/apache/iceberg/pull/3947#issuecomment-1019071181


   @rdblue: We have a corresponding change in the Hive code where during the 
migration of an external Hive table to Iceberg table we would like to set the 
"schema.name-mapping.default", so Impala could read the migrated table.
   
   We can do in a different way:
   - When Iceberg rewrites the schema, it should rewrite the 
`schema.name-mapping.default` value too - This might be a more general solution 
than depending on the Hive schema generation creating the same ids as the 
Iceberg schema generation. On the other hand I am not sure this worth the added 
complexity.
   
   What are your thoughts Ryan?


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