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

Abyss-lord · Abyss-lord · commit 1d1b13c2f618 · 2025-01-15T20:28:05.000+08:00
fix conflict.
diff --git a/core/src/main/java/org/apache/gravitino/connector/BaseCatalog.java b/core/src/main/java/org/apache/gravitino/connector/BaseCatalog.java
@@ -209,6 +209,8 @@ public void initAuthorizationPluginInstance(IsolatedClassLoader classLoader) {
           }
           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.
             authorizationPlugin =
                 classLoader.withClassLoader(
                     cl ->

Original file line number	Diff line number	Diff line change
`@@ -209,6 +209,8 @@ public void initAuthorizationPluginInstance(IsolatedClassLoader classLoader) {`
`209`	`209`	`}`
`210`	`210`	`try (BaseAuthorization<?> authorization =`
`211`	`211`	`BaseAuthorization.createAuthorization(classLoader, authorizationProvider)) {`
	`212`	`+ // Load the authorization plugin with the class loader of the catalog.`
	`213`	`+ // Because the JDBC authorization plugin may load JDBC driver using the class loader.`
`212`	`214`	`authorizationPlugin =`
`213`	`215`	`classLoader.withClassLoader(`
`214`	`216`	`cl ->`