[#2479] refactor: separate API for client using: SupportsMetalakes and SupportsCatalogs (#3390)

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

Today Gravitino java client and server sides share the same interfaces,
like SupportsMetalakes, SupportsCatalogs, SupportsSchemas, etc. These
interfaces are good for server side, but not good for client side. After
some discussion with Jerry and others, if we want to make it easier to
use, and still keep server side code stable, the only way is to separate
the APIs: create individual APIs for the java client.

This PR will introduce the simplified API for client: SupportsMetalakes
and SupportsCatalogs (in the package
com.datastrato.gravitino.client.api). Their method signatures have been
changed, so the client code will be clear. The integration tests which
uses Java client to manipulate metadata also be updated. (If we don't
want to use the same classname, I can refactor them later in a batch)

### Why are the changes needed?

To make the client API simple and easy to use. 

Fix: #2479

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

Will change the Java client API, mainly on the method signatures; for
example, "listCatalogs()" won't need an input parameter, "loadCatalog()"
will use a String value as the input parameter instead of a
NameIdentifier object, etc.

### How was this patch tested?

All existing integration test will cover the API change.

Author: shaofengshi

api/src/main/java/com/datastrato/gravitino/rel/SupportsCatalogs.java

@@ -0,0 +1,107 @@
+/*
+ * Copyright 2024 Datastrato Pvt Ltd.
+ * This software is licensed under the Apache License version 2.
+ */
+package com.datastrato.gravitino.rel;
+
+import com.datastrato.gravitino.Catalog;
+import com.datastrato.gravitino.CatalogChange;
+import com.datastrato.gravitino.CatalogProvider;
+import com.datastrato.gravitino.NameIdentifier;
+import com.datastrato.gravitino.annotation.Evolving;
+import com.datastrato.gravitino.exceptions.CatalogAlreadyExistsException;
+import com.datastrato.gravitino.exceptions.NoSuchCatalogException;
+import com.datastrato.gravitino.exceptions.NoSuchMetalakeException;
+import java.util.Map;
+
+/**
+ * Client interface for supporting catalogs. It includes methods for listing, loading, creating,
+ * altering and dropping catalogs.
+ */
+@Evolving
+public interface SupportsCatalogs {
+
+  /**
+   * List all catalogs in the metalake.
+   *
+   * @return The list of catalog's name identifiers.
+   * @throws NoSuchMetalakeException If the metalake with namespace does not exist.
+   */
+  NameIdentifier[] listCatalogs() throws NoSuchMetalakeException;
+
+  /**
+   * List all catalogs with their information in the metalake.
+   *
+   * @return The list of catalog's information.
+   * @throws NoSuchMetalakeException If the metalake with namespace does not exist.
+   */
+  Catalog[] listCatalogsInfo() throws NoSuchMetalakeException;
+
+  /**
+   * Load a catalog by its identifier.
+   *
+   * @param catalogName the identifier of the catalog.
+   * @return The catalog.
+   * @throws NoSuchCatalogException If the catalog does not exist.
+   */
+  Catalog loadCatalog(String catalogName) throws NoSuchCatalogException;
+
+  /**
+   * Check if a catalog exists.
+   *
+   * @param catalogName The identifier of the catalog.
+   * @return True if the catalog exists, false otherwise.
+   */
+  default boolean catalogExists(String catalogName) {
+    try {
+      loadCatalog(catalogName);
+      return true;
+    } catch (NoSuchCatalogException e) {
+      return false;
+    }
+  }
+
+  /**
+   * Create a catalog with specified identifier.
+   *
+   * <p>The parameter "provider" is a short name of the catalog, used to tell Gravitino which
+   * catalog should be created. The short name should be the same as the {@link CatalogProvider}
+   * interface provided.
+   *
+   * @param catalogName the name of the catalog.
+   * @param type the type of the catalog.
+   * @param provider the provider of the catalog.
+   * @param comment the comment of the catalog.
+   * @param properties the properties of the catalog.
+   * @return The created catalog.
+   * @throws NoSuchMetalakeException If the metalake does not exist.
+   * @throws CatalogAlreadyExistsException If the catalog already exists.
+   */
+  Catalog createCatalog(
+      String catalogName,
+      Catalog.Type type,
+      String provider,
+      String comment,
+      Map<String, String> properties)
+      throws NoSuchMetalakeException, CatalogAlreadyExistsException;
+
+  /**
+   * Alter a catalog with specified identifier.
+   *
+   * @param catalogName the identifier of the catalog.
+   * @param changes the changes to apply to the catalog.
+   * @return The altered catalog.
+   * @throws NoSuchCatalogException If the catalog does not exist.
+   * @throws IllegalArgumentException If the changes cannot be applied to the catalog.
+   */
+  Catalog alterCatalog(String catalogName, CatalogChange... changes)
+      throws NoSuchCatalogException, IllegalArgumentException;
+
+  /**
+   * Drop a catalog with specified identifier.
+   *
+   * @param catalogName the name of the catalog.
+   * @return True if the catalog was dropped, false otherwise.
+   */
+  boolean dropCatalog(String catalogName);
+}

api/src/main/java/com/datastrato/gravitino/rel/SupportsMetalakes.java

@@ -0,0 +1,83 @@
+/*
+ * Copyright 2024 Datastrato Pvt Ltd.
+ * This software is licensed under the Apache License version 2.
+ */
+package com.datastrato.gravitino.rel;
+
+import com.datastrato.gravitino.Metalake;
+import com.datastrato.gravitino.MetalakeChange;
+import com.datastrato.gravitino.annotation.Evolving;
+import com.datastrato.gravitino.exceptions.MetalakeAlreadyExistsException;
+import com.datastrato.gravitino.exceptions.NoSuchMetalakeException;
+import java.util.Map;
+
+/**
+ * Client interface for supporting metalakes. It includes methods for listing, loading, creating,
+ * altering and dropping metalakes.
+ */
+@Evolving
+public interface SupportsMetalakes {
+
+  /**
+   * List all metalakes.
+   *
+   * @return The list of metalakes.
+   */
+  Metalake[] listMetalakes();
+
+  /**
+   * Load a metalake by its identifier.
+   *
+   * @param name the name of the metalake.
+   * @return The metalake.
+   * @throws NoSuchMetalakeException If the metalake does not exist.
+   */
+  Metalake loadMetalake(String name) throws NoSuchMetalakeException;
+
+  /**
+   * Check if a metalake exists.
+   *
+   * @param name The name of the metalake.
+   * @return True if the metalake exists, false otherwise.
+   */
+  default boolean metalakeExists(String name) {
+    try {
+      loadMetalake(name);
+      return true;
+    } catch (NoSuchMetalakeException e) {
+      return false;
+    }
+  }
+
+  /**
+   * Create a metalake with specified identifier.
+   *
+   * @param name The name of the metalake.
+   * @param comment The comment of the metalake.
+   * @param properties The properties of the metalake.
+   * @return The created metalake.
+   * @throws MetalakeAlreadyExistsException If the metalake already exists.
+   */
+  Metalake createMetalake(String name, String comment, Map<String, String> properties)
+      throws MetalakeAlreadyExistsException;
+
+  /**
+   * Alter a metalake with specified identifier.
+   *
+   * @param name The name of the metalake.
+   * @param changes The changes to apply.
+   * @return The altered metalake.
+   * @throws NoSuchMetalakeException If the metalake does not exist.
+   * @throws IllegalArgumentException If the changes cannot be applied to the metalake.
+   */
+  Metalake alterMetalake(String name, MetalakeChange... changes)
+      throws NoSuchMetalakeException, IllegalArgumentException;
+
+  /**
+   * Drop a metalake with specified identifier.
+   *
+   * @param name The identifier of the metalake.
+   * @return True if the metalake was dropped, false otherwise.
+   */
+  boolean dropMetalake(String name);
+}

catalogs/catalog-hadoop/src/test/java/com/datastrato/gravitino/catalog/hadoop/integration/test/HadoopCatalogIT.java

@@ -67,7 +67,7 @@ public static void setup() throws IOException {
 
   @AfterAll
   public static void stop() throws IOException {
-    client.dropMetalake(NameIdentifier.of(metalakeName));
+    client.dropMetalake(metalakeName);
 
     if (hdfs != null) {
       hdfs.close();
@@ -85,22 +85,18 @@ private static void createMetalake() {
     Assertions.assertEquals(0, gravitinoMetalakes.length);
 
     GravitinoMetalake createdMetalake =
-        client.createMetalake(NameIdentifier.of(metalakeName), "comment", Collections.emptyMap());
-    GravitinoMetalake loadMetalake = client.loadMetalake(NameIdentifier.of(metalakeName));
+        client.createMetalake(metalakeName, "comment", Collections.emptyMap());
+    GravitinoMetalake loadMetalake = client.loadMetalake(metalakeName);
     Assertions.assertEquals(createdMetalake, loadMetalake);
 
     metalake = loadMetalake;
   }
 
   private static void createCatalog() {
     metalake.createCatalog(
-        NameIdentifier.of(metalakeName, catalogName),
-        Catalog.Type.FILESET,
-        provider,
-        "comment",
-        ImmutableMap.of());
+        catalogName, Catalog.Type.FILESET, provider, "comment", ImmutableMap.of());
 
-    catalog = metalake.loadCatalog(NameIdentifier.of(metalakeName, catalogName));
+    catalog = metalake.loadCatalog(catalogName);
   }
 
   private static void createSchema() {
@@ -547,11 +543,7 @@ public void testDropCatalogWithEmptySchema() {
     // Create a catalog without specifying location.
     Catalog filesetCatalog =
         metalake.createCatalog(
-            NameIdentifier.of(metalakeName, catalogName),
-            Catalog.Type.FILESET,
-            provider,
-            "comment",
-            ImmutableMap.of());
+            catalogName, Catalog.Type.FILESET, provider, "comment", ImmutableMap.of());
 
     // Create a schema without specifying location.
     String schemaName =
@@ -574,11 +566,9 @@ public void testDropCatalogWithEmptySchema() {
         "schema should not be exists");
 
     // Drop the catalog.
-    dropped = metalake.dropCatalog(NameIdentifier.of(metalakeName, catalogName));
+    dropped = metalake.dropCatalog(catalogName);
     Assertions.assertTrue(dropped, "catalog should be dropped");
-    Assertions.assertFalse(
-        metalake.catalogExists(NameIdentifier.of(metalakeName, catalogName)),
-        "catalog should not be exists");
+    Assertions.assertFalse(metalake.catalogExists(catalogName), "catalog should not be exists");
   }
 
   private Fileset createFileset(