yuqi1129 commented on PR #3852:
URL: https://github.com/apache/gravitino/pull/3852#issuecomment-2222704271
@jerqi
I hesitate to remove the catalog proxy and use a new proxy to unify catalog,
schema and fileset authentication as there seem to be some customized logic for
some operations such as
```
try {
SchemaEntity schemaEntity =
hadoopCatalogOperations
.getStore()
.get(ident, Entity.EntityType.SCHEMA, SchemaEntity.class);
Map<String, String> properties =
Optional.ofNullable(schemaEntity.properties()).orElse(Collections.emptyMap());
// We need the logic mentioned above to obtain the properties, it's not
appropriate to move it to a proxy class.
// Reset the current user based on the name identifier.
UserGroupInformation user = getUGIByIdent(properties, ident);
boolean r = doAs(user, () -> hadoopCatalogOperations.dropSchema(ident,
cascade), ident);
cleanUserInfo(ident);
return r;
} catch (IOException ioe) {
throw new RuntimeException("Failed to delete schema " + ident, ioe);
}
```
--
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]
