jarredhj0214 commented on code in PR #13562:
URL: https://github.com/apache/gravitino/pull/13562#discussion_r4237863395
##########
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:
Added in 3d0f59800: both Python client entry points now support
include_properties=True (the default) and False for lightweight descriptors.
Updated the Python docs example and added request/delegation and descriptor
regression tests. All 22 targeted Python tests, Black/Pylint checks, and
:docs:build pass.
--
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]
