jjiang037 commented on PR #5770: URL: https://github.com/apache/hive/pull/5770#issuecomment-2802471019
Hi @Aggarwal-Raghav, thanks for the review and question! You're right—HIVE-27285 introduced ownership info (ownerName, ownerType) in TableMeta and updated filterTableMetas to work with it. But it didn’t use that info within the TableFilterContext. This PR (HIVE-28898) addresses that gap: Wires ownership into TableFilterContext: Adds a createFromTableMetas factory method that converts TableMeta to Table and copies ownerName and ownerType. Populates HivePrivilegeObjects properly: getInputHObjs now includes owner info when building the list, enabling plugins like Ranger to use it. Ensures correct flow in HMSHandler: Updates get_all_tables to fetch TableMeta (with ownership) before filtering, so it propagates down properly. In short: HIVE-27285 exposed ownership data; this PR makes sure it’s actually used during filtering and authorization. -- 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: gitbox-unsubscr...@hive.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org For additional commands, e-mail: gitbox-h...@hive.apache.org
