deniskuzZ commented on code in PR #5947:
URL: https://github.com/apache/hive/pull/5947#discussion_r2202709219
##########
standalone-metastore/metastore-common/src/main/java/org/apache/hadoop/hive/metastore/IMetaStoreClient.java:
##########
@@ -189,34 +189,6 @@ List<String> getDatabases(String catName, String
databasePattern)
*/
List<String> getAllDatabases(String catName) throws MetaException,
TException;
- /**
- * Get the names of all tables in the specified database that satisfy the
supplied
- * table name pattern.
- * @param dbName database name.
- * @param tablePattern pattern for table name to conform to
- * @return List of table names.
- * @throws MetaException error fetching information from the RDBMS
- * @throws TException thrift transport error
- * @throws UnknownDBException indicated database to search in does not exist.
- */
- List<String> getTables(String dbName, String tablePattern)
Review Comment:
@ramitg254, we can't just go ahead and change the client API. If we have an
agreement that this could be dropped , we should first deprecate it
cc @dengzhhu653
--
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
