hdygxsj commented on code in PR #6842:
URL: https://github.com/apache/gravitino/pull/6842#discussion_r2051904667
##########
server/src/main/java/org/apache/gravitino/server/GravitinoServer.java:
##########
@@ -90,6 +91,8 @@ public void initialize() {
ServerAuthenticator.getInstance().initialize(serverConfig);
+ GravitinoAuthorizerProvider.getInstance().initialize(serverConfig);
Review Comment:
Unless we check whether Gravitino has enabled metadata authorization for
every list API method, the process will inevitably enter the
`MetadataFilterHelper` when filtering the results of the list API. The current
plan is to add a default implementation to `GravitinoAuthorizer` that bypasses
authorization checks. This way, we can avoid having to check whether Gravitino
has enabled metadata authorization every time a list method is executed.
--
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]
