JingsongLi commented on code in PR #8179:
URL: https://github.com/apache/paimon/pull/8179#discussion_r3386265295
##########
paimon-core/src/main/java/org/apache/paimon/catalog/DelegateCatalog.java:
##########
@@ -364,6 +366,35 @@ public void alterFunction(
wrapped.alterFunction(identifier, changes, ignoreIfNotExists);
}
+ @Override
+ public List<String> listResources(String databaseName) throws
DatabaseNotExistException {
Review Comment:
Please delegate the new paged resource APIs here as well. RESTCatalog
implements listResourcesPaged, listResourceDetailsPaged, and
listResourcesPagedGlobally, but CatalogFactory wraps normal catalogs in
CachingCatalog and sometimes PrivilegedCatalog, both via DelegateCatalog.
Through those wrappers the Catalog defaults are used instead, so
paging/filtering is ignored, resource details return empty, and global resource
listing throws UnsupportedOperationException. Add the three forwarding
overrides here, mirroring the existing table/view/function paging methods.
--
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]
