[ https://issues.apache.org/jira/browse/HIVE-26148?focusedWorklogId=764878&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-764878 ]
ASF GitHub Bot logged work on HIVE-26148: ----------------------------------------- Author: ASF GitHub Bot Created on: 02/May/22 08:45 Start Date: 02/May/22 08:45 Worklog Time Spent: 10m Work Description: pvary commented on code in PR #3218: URL: https://github.com/apache/hive/pull/3218#discussion_r862684011 ########## standalone-metastore/metastore-server/src/test/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClientPreCatalog.java: ########## @@ -1296,7 +1299,7 @@ public List<String> getDatabases(String databasePattern) @Override public List<String> getAllDatabases() throws MetaException { try { - return filterHook.filterDatabases(client.get_all_databases()); + return filterHook.filterDatabases(null, client.get_all_databases()); Review Comment: Should this be `getDefaultCatalog(conf)` instead off null? Issue Time Tracking ------------------- Worklog Id: (was: 764878) Time Spent: 2.5h (was: 2h 20m) > 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: 2.5h > 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)