laserninja commented on code in PR #10671:
URL: https://github.com/apache/gravitino/pull/10671#discussion_r3296179042
##########
iceberg/iceberg-rest-server/src/main/java/org/apache/gravitino/iceberg/service/rest/IcebergNamespaceOperations.java:
##########
@@ -114,7 +116,8 @@ public Response listNamespaces(
IcebergRequestContext context =
new IcebergRequestContext(httpServletRequest(), catalogName);
ListNamespacesResponse response =
- namespaceOperationDispatcher.listNamespaces(context,
parentNamespace);
+ namespaceOperationDispatcher.listNamespaces(
+ context, parentNamespace, pageToken, pageSize);
Review Comment:
pagination now runs after authorization filtering in all three REST
endpoints (IcebergNamespaceOperations, IcebergTableOperations,
IcebergViewOperations). The flow is: dispatcher returns full list → authz
filter applied → IcebergPaginationHelper paginates the filtered result. This
ensures pageSize/nextPageToken are consistent with the authorized view.
--
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]
