yuqi1129 commented on code in PR #12873:
URL: https://github.com/apache/gravitino/pull/12873#discussion_r3924347491
##########
core/src/main/java/org/apache/gravitino/hook/FunctionHookDispatcher.java:
##########
@@ -108,6 +110,11 @@ public Function alterFunction(NameIdentifier ident,
FunctionChange... changes)
@Override
public boolean dropFunction(NameIdentifier ident) {
- return dispatcher.dropFunction(ident);
+ boolean dropped = dispatcher.dropFunction(ident);
+ if (dropped) {
+ AuthorizationUtils.authorizationPluginRemovePrivileges(
+ ident, Entity.EntityType.FUNCTION, Collections.emptyList());
+ }
Review Comment:
Fixed in 5541831998. FunctionHookDispatcher now applies the same
function/schema case-sensitivity normalization as FunctionNormalizeDispatcher
before invalidating authorization state, so non-canonical caller input targets
the canonical cache key. The normalization is shared through CapabilityHelpers,
and the new case-insensitive test covers successful, no-op, and exceptional
drop paths.
--
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]
