[GitHub] [incubator-paimon] SteNicholas commented on a diff in pull request #1185: [Feature] [Java-api][Catalog]Get all the table information under catalog

["SteNicholas (via GitHub)" <gi...@apache.org>]

SteNicholas commented on code in PR #1185:
URL: https://github.com/apache/incubator-paimon/pull/1185#discussion_r1203318955


##########
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> listAllTables() {

Review Comment:
   ```suggestion
       default List<String> listTables() {
   ```



-- 
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: issues-unsubscribe@paimon.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org