[GitHub] [iceberg] gaborkaszab commented on pull request #5763: Core, Hive-metastore: Table owner can be incorrect in HMS

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

gaborkaszab commented on PR #5763:
URL: https://github.com/apache/iceberg/pull/5763#issuecomment-1276245441

   > In PyIceberg we had a similar concern. We ended up by fetching username from the properties: https://github.com/apache/iceberg/blob/master/python/pyiceberg/catalog/hive.py#L317 Similar to what we do for location (`METADATA_LOCATION_PROP`).
   
   Thanks for taking a look, @Fokko ! For some reason I figured that using a table property to provide the owner is not the proper way to go. However, I changed my patch to use a table property as you suggested. This frankly seems a way simpler implementation.


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