[apache#3295] fix(catalog-hive):The Hive Catalog Bug In Multiple Kerb…

…erized HMS
theoryxu · May 10, 2024 · c74aeab · c74aeab
1 parent bd28288
commit c74aeab
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/...talog-hive/src/main/java/com/datastrato/gravitino/catalog/hive/HiveCatalogOperations.java b/...talog-hive/src/main/java/com/datastrato/gravitino/catalog/hive/HiveCatalogOperations.java
@@ -106,9 +106,7 @@ public class HiveCatalogOperations implements CatalogOperations, SupportsSchemas
   public static final Map<String, String> GRAVITINO_CONFIG_TO_HIVE =
       ImmutableMap.of(
           METASTORE_URIS,
-          ConfVars.METASTOREURIS.varname,
-          PRINCIPAL,
-          ConfVars.METASTORE_KERBEROS_PRINCIPAL.varname);
+          ConfVars.METASTOREURIS.varname);
 
   /**
    * Initializes the Hive catalog operations with the provided configuration.

diff --git a/docs/apache-hive-catalog.md b/docs/apache-hive-catalog.md
@@ -35,7 +35,7 @@ The Hive catalog supports creating, updating, and deleting databases and tables
 | `gravitino.bypass.`                      | Property name with this prefix passed down to the underlying HMS client for use. Such as `gravitino.bypass.hive.metastore.failure.retries = 3` indicate 3 times of retries upon failure of Thrift metastore calls | (none)        | No                           | 0.2.0         |
 | `client.pool-cache.eviction-interval-ms` | The cache pool eviction interval.                                                                                                                                                                                 | 300000        | No                           | 0.4.0         |
 | `impersonation-enable`                   | Enable user impersonation for Hive catalog.                                                                                                                                                                       | false         | No                           | 0.4.0         |
-| `kerberos.principal`                     | The Kerberos principal for the catalog. You should configure `gravitino.bypass.hadoop.security.authentication` and `gravitino.bypass.hive.metastore.sasl.enabled`if you want to use Kerberos.                     | (none)        | required if you use kerberos | 0.4.0         |
+| `kerberos.principal`                     | The Kerberos principal for the catalog. You should configure `gravitino.bypass.hadoop.security.authentication`, `gravitino.bypass.hive.metastore.kerberos.principal` and `gravitino.bypass.hive.metastore.sasl.enabled`if you want to use Kerberos.                   | (none)        | required if you use kerberos | 0.4.0         |
 | `kerberos.keytab-uri`                    | The uri of key tab for the catalog. Now supported protocols are `https`, `http`, `ftp`, `file`.                                                                                                                   | (none)        | required if you use kerberos | 0.4.0         |
 | `kerberos.check-interval-sec`            | The interval to check validness of the principal                                                                                                                                                                  | 60            | No                           | 0.4.0         |
 | `kerberos.keytab-fetch-timeout-sec`      | The timeout to fetch key tab                                                                                                                                                                                      | 60            | No                           | 0.4.0         |