mchades commented on code in PR #11028:
URL: https://github.com/apache/gravitino/pull/11028#discussion_r3264073144
##########
authorizations/authorization-ranger/src/test/java/org/apache/gravitino/authorization/ranger/integration/test/RangerFilesetIT.java:
##########
Review Comment:
Are these changes in this file necessary? It seems unrelated to this PR?
##########
core/src/main/java/org/apache/gravitino/authorization/AuthorizationUtils.java:
##########
@@ -602,6 +602,10 @@ public static List<String> getMetadataObjectLocation(
case TOPIC:
// Topic doesn't have locations now.
break;
+ case VIEW:
+ // Views are logical metadata objects without a single storage
location; privilege plugins
+ // operate on metadata only (same idea as TOPIC).
+ break;
Review Comment:
can move this change to another PR?
--
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]
