[apache#6236] fix(core): fix possible resource leak in BaseCatalog

fix possible resource leak in BaseCatalog.
Abyss-lord · Jan 15, 2025 · b669a0b · b669a0b
1 parent 857a464
commit b669a0b
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/core/src/main/java/org/apache/gravitino/connector/BaseCatalog.java b/core/src/main/java/org/apache/gravitino/connector/BaseCatalog.java
@@ -205,12 +205,8 @@ public void initAuthorizationPluginInstance(IsolatedClassLoader classLoader) {
             LOG.info("Authorization provider is not set!");
             return;
           }
-          try {
-            BaseAuthorization<?> authorization =
-                BaseAuthorization.createAuthorization(classLoader, authorizationProvider);
-
-            // Load the authorization plugin with the class loader of the catalog.
-            // Because the JDBC authorization plugin may load JDBC driver using the class loader.
+          try (BaseAuthorization<?> authorization =
+              BaseAuthorization.createAuthorization(classLoader, authorizationProvider)) {
             authorizationPlugin =
                 classLoader.withClassLoader(
                     cl ->