jerqi commented on code in PR #5096:
URL: https://github.com/apache/gravitino/pull/5096#discussion_r1798815399
##########
core/src/main/java/org/apache/gravitino/hook/CatalogHookDispatcher.java:
##########
@@ -71,6 +72,10 @@ public Catalog createCatalog(
String comment,
Map<String, String> properties)
throws NoSuchMetalakeException, CatalogAlreadyExistsException {
+ // Check whether the current user exists or not
+ AuthorizationUtils.checkCurrentUser(
Review Comment:
We should check every operations actually.
We don't support meta authentication now. I think this should be included to
meta authentication.
But for some managers, we don't a dispatcher interface for them like
ownerManager and TagManager. We will add the interfaces first. I'm not sure to
do them now. So I just make a minor improvement to check `createXXX` operation
first.
--
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]
