yuqi1129 commented on code in PR #13562:
URL: https://github.com/apache/gravitino/pull/13562#discussion_r4237502361


##########
api/src/main/java/org/apache/gravitino/SupportsCatalogs.java:
##########
@@ -53,6 +53,26 @@ public interface SupportsCatalogs {
    */
   Catalog[] listCatalogsInfo() throws NoSuchMetalakeException;
 
+  /**
+   * List all catalogs with their information in the metalake.
+   *
+   * <p>When {@code includeProperties} is {@code false}, catalog properties 
are omitted and {@link
+   * Catalog#properties()} returns an empty map, never {@code null}. All 
properties, including
+   * {@code in-use}, are absent, so the result cannot be distinguished from a 
catalog with no
+   * properties. Name, type, provider, comment, and audit information are 
still populated.
+   *
+   * <p>The default implementation throws {@link 
UnsupportedOperationException}.
+   *
+   * @param includeProperties whether to include resolved catalog properties
+   * @return The list of catalog's information.
+   * @throws NoSuchMetalakeException If the metalake does not exist.
+   * @throws UnsupportedOperationException If the implementation does not 
support this overload.
+   */
+  default Catalog[] listCatalogsInfo(boolean includeProperties) throws 
NoSuchMetalakeException {

Review Comment:
   Have you changed the Python client here?



-- 
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]

Reply via email to