[apache#3673] fix(core): avoid to initialize catalog ops when closing…

… catalog (apache#3672)

### What changes were proposed in this pull request?

avoid to initialize catalog ops when closing catalog

### Why are the changes needed?

Fix: apache#3673 

### Does this PR introduce _any_ user-facing change?

no

### How was this patch tested?

no

core/src/main/java/com/datastrato/gravitino/catalog/CatalogManager.java

@@ -162,7 +162,7 @@ public void close() {
         classLoader.withClassLoader(
             cl -> {
               if (catalog != null) {
-                catalog.ops().close();
+                catalog.close();
               }
               catalog = null;
               return null;

core/src/main/java/com/datastrato/gravitino/connector/BaseCatalog.java

@@ -13,6 +13,8 @@
 import com.google.common.annotations.VisibleForTesting;
 import com.google.common.base.Preconditions;
 import com.google.common.collect.Maps;
+import java.io.Closeable;
+import java.io.IOException;
 import java.util.Map;
 import java.util.Optional;
 import org.slf4j.Logger;
@@ -32,7 +34,7 @@
  */
 @Evolving
 public abstract class BaseCatalog<T extends BaseCatalog>
-    implements Catalog, CatalogProvider, HasPropertyMetadata {
+    implements Catalog, CatalogProvider, HasPropertyMetadata, Closeable {
   private static final Logger LOG = LoggerFactory.getLogger(BaseCatalog.class);
 
   // This variable is used as a key in properties of catalogs to inject custom operation to
@@ -153,6 +155,14 @@ public CatalogOperations ops() {
     return ops;
   }
 
+  @Override
+  public void close() throws IOException {
+    if (ops != null) {
+      ops.close();
+      ops = null;
+    }
+  }
+
   public Capability capability() {
     if (capability == null) {
       synchronized (this) {