[ https://issues.apache.org/jira/browse/HIVE-26148?focusedWorklogId=764052&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-764052 ]
ASF GitHub Bot logged work on HIVE-26148: ----------------------------------------- Author: ASF GitHub Bot Created on: 29/Apr/22 07:58 Start Date: 29/Apr/22 07:58 Worklog Time Spent: 10m Work Description: pvary commented on code in PR #3218: URL: https://github.com/apache/hive/pull/3218#discussion_r861552317 ########## ql/src/java/org/apache/hadoop/hive/ql/security/authorization/PrivilegeSynchronizer.java: ########## @@ -170,7 +170,8 @@ private void addGrantPrivilegesToBag(HivePolicyProvider policyProvider, Privileg case COLUMN: objectAcls = policyProvider - .getResourceACLs(new HivePrivilegeObject(HivePrivilegeObjectType.COLUMN, dbName, objName, null, columnName)); + .getResourceACLs(new HivePrivilegeObject(HivePrivilegeObjectType.COLUMN, dbName, objName, null, Review Comment: Shouldn't we need a catalog name here? Issue Time Tracking ------------------- Worklog Id: (was: 764052) Time Spent: 40m (was: 0.5h) > Keep MetaStoreFilterHook interface compatibility after introducing catalogs > --------------------------------------------------------------------------- > > Key: HIVE-26148 > URL: https://issues.apache.org/jira/browse/HIVE-26148 > Project: Hive > Issue Type: Improvement > Components: Hive > Affects Versions: 3.0.0 > Reporter: Wechar > Assignee: Wechar > Priority: Minor > Labels: pull-request-available > Fix For: 4.0.0-alpha-1 > > Time Spent: 40m > Remaining Estimate: 0h > > Hive 3.0 introduce catalog concept, when we upgrade hive dependency version > from 2.3 to 3.x, we found some interfaces of *MetaStoreFilterHook* are not > compatible: > {code:bash} > git show ba8a99e115 -- > standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/MetaStoreFilterHook.java > {code} > {code:bash} > --- > a/standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/MetaStoreFilterHook.java > +++ > b/standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/MetaStoreFilterHook.java > /** > * Filter given list of tables > - * @param dbName > - * @param tableList > + * @param catName catalog name > + * @param dbName database name > + * @param tableList list of table returned by the metastore > * @return List of filtered table names > */ > - public List<String> filterTableNames(String dbName, List<String> > tableList) throws MetaException; > + List<String> filterTableNames(String catName, String dbName, List<String> > tableList) > + throws MetaException; > {code} > We can remain the previous interfaces and use the default catalog to > implement. -- This message was sent by Atlassian Jira (v8.20.7#820007)