liugddx commented on code in PR #1185:
URL: https://github.com/apache/incubator-paimon/pull/1185#discussion_r1199766131
##########
paimon-core/src/main/java/org/apache/paimon/catalog/Catalog.java:
##########
@@ -108,6 +111,31 @@ void dropDatabase(String name, boolean ignoreIfNotExists,
boolean cascade)
*/
List<String> listTables(String databaseName) throws
DatabaseNotExistException;
+ /**
+ * Get names of all tables under this catalog. An empty list is returned
if none exists.
+ *
+ * <p>NOTE: System tables will not be listed.
+ *
+ * @return a list of the names of all tables in this catalog
+ */
+ default List<Identifier> listTables() {
Review Comment:
> This interface appears to be confused. List tables from current database
or list all tables?
I changed it to `listAllTables`, Is this okay? I want to get all the tables
under the catalog.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
