jerryshao commented on code in PR #3946: URL: https://github.com/apache/gravitino/pull/3946#discussion_r1671762074
########## core/src/main/java/com/datastrato/gravitino/connector/BaseCatalog.java: ########## @@ -57,6 +65,9 @@ public abstract class BaseCatalog<T extends BaseCatalog> // The object you used is not stable, don't use it unless you know what you are doing. @VisibleForTesting public static final String CATALOG_OPERATION_IMPL = "ops-impl"; + // Underlying access control system plugin for this catalog. + private volatile BaseAuthorization<?> authorization; Review Comment: The concern is that each time when you use this `authorization` you have to check if it is null or not, so it may be not so convenient. -- 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]
