[GitHub] [iceberg] flyrain commented on pull request #3914: Hive: Add table UUID to HMS table properties.

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

flyrain commented on pull request #3914:
URL: https://github.com/apache/iceberg/pull/3914#issuecomment-1015653969


   > As per our previous discussions, in the long term we would like to keep Iceberg table properties and HMS table properties in sync. With this change and with the Hive 4.0.0 changes in place the UUID will be synchronised back to the Iceberg table properties too. Would this be a problem?
   
   UUID will be considered as immutable, read-only in HMS. It isn't needed to be synced back. I'd be nice if there is a way to label it read-only in HMS. Not sure if that is possible. 
   
   Hi @aokolnychyi, @RussellSpitzer, @szehon-ho, @anuragmantri  can you take a look at this PR?


-- 
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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org