This is an automated email from the ASF dual-hosted git repository.
lzljs3620320 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/paimon.git
The following commit(s) were added to refs/heads/master by this push:
new 11cebd43fc [hive] Add metastore.client.class option to hive catalog
configuration document. (#4627)
11cebd43fc is described below
commit 11cebd43fc2bddb0fb782e1f25e3e276b808c8b7
Author: Kerwin <[email protected]>
AuthorDate: Tue Dec 3 23:43:27 2024 +0800
[hive] Add metastore.client.class option to hive catalog configuration
document. (#4627)
---
.../shortcodes/generated/hive_catalog_configuration.html | 7 +++++++
.../src/main/java/org/apache/paimon/hive/HiveCatalog.java | 2 +-
.../main/java/org/apache/paimon/hive/HiveCatalogFactory.java | 11 -----------
.../main/java/org/apache/paimon/hive/HiveCatalogOptions.java | 9 +++++++++
4 files changed, 17 insertions(+), 12 deletions(-)
diff --git a/docs/layouts/shortcodes/generated/hive_catalog_configuration.html
b/docs/layouts/shortcodes/generated/hive_catalog_configuration.html
index e0257d301b..7b6242616f 100644
--- a/docs/layouts/shortcodes/generated/hive_catalog_configuration.html
+++ b/docs/layouts/shortcodes/generated/hive_catalog_configuration.html
@@ -65,5 +65,12 @@ If you don't want to access the location by the filesystem
of hive when using a
you can set this option to true.
</td>
</tr>
+ <tr>
+ <td><h5>metastore.client.class</h5></td>
+ <td style="word-wrap:
break-word;">"org.apache.hadoop.hive.metastore.HiveMetaStoreClient"</td>
+ <td>String</td>
+ <td>Class name of Hive metastore client.
+NOTE: This class must directly implements
org.apache.hadoop.hive.metastore.IMetaStoreClient.</td>
+ </tr>
</tbody>
</table>
diff --git
a/paimon-hive/paimon-hive-catalog/src/main/java/org/apache/paimon/hive/HiveCatalog.java
b/paimon-hive/paimon-hive-catalog/src/main/java/org/apache/paimon/hive/HiveCatalog.java
index 5157e60600..151e2b4d2c 100644
---
a/paimon-hive/paimon-hive-catalog/src/main/java/org/apache/paimon/hive/HiveCatalog.java
+++
b/paimon-hive/paimon-hive-catalog/src/main/java/org/apache/paimon/hive/HiveCatalog.java
@@ -1349,7 +1349,7 @@ public class HiveCatalog extends AbstractCatalog {
return new HiveCatalog(
fileIO,
hiveConf,
- options.get(HiveCatalogFactory.METASTORE_CLIENT_CLASS),
+ options.get(HiveCatalogOptions.METASTORE_CLIENT_CLASS),
options,
warehouse.toUri().toString());
}
diff --git
a/paimon-hive/paimon-hive-catalog/src/main/java/org/apache/paimon/hive/HiveCatalogFactory.java
b/paimon-hive/paimon-hive-catalog/src/main/java/org/apache/paimon/hive/HiveCatalogFactory.java
index 95da003716..eff06831dd 100644
---
a/paimon-hive/paimon-hive-catalog/src/main/java/org/apache/paimon/hive/HiveCatalogFactory.java
+++
b/paimon-hive/paimon-hive-catalog/src/main/java/org/apache/paimon/hive/HiveCatalogFactory.java
@@ -21,23 +21,12 @@ package org.apache.paimon.hive;
import org.apache.paimon.catalog.Catalog;
import org.apache.paimon.catalog.CatalogContext;
import org.apache.paimon.catalog.CatalogFactory;
-import org.apache.paimon.options.ConfigOption;
-import org.apache.paimon.options.ConfigOptions;
import static org.apache.paimon.hive.HiveCatalogOptions.IDENTIFIER;
/** Factory to create {@link HiveCatalog}. */
public class HiveCatalogFactory implements CatalogFactory {
- public static final ConfigOption<String> METASTORE_CLIENT_CLASS =
- ConfigOptions.key("metastore.client.class")
- .stringType()
-
.defaultValue("org.apache.hadoop.hive.metastore.HiveMetaStoreClient")
- .withDescription(
- "Class name of Hive metastore client.\n"
- + "NOTE: This class must directly
implements "
- +
"org.apache.hadoop.hive.metastore.IMetaStoreClient.");
-
@Override
public String identifier() {
return IDENTIFIER;
diff --git
a/paimon-hive/paimon-hive-catalog/src/main/java/org/apache/paimon/hive/HiveCatalogOptions.java
b/paimon-hive/paimon-hive-catalog/src/main/java/org/apache/paimon/hive/HiveCatalogOptions.java
index 38f73bc6bd..ceab498368 100644
---
a/paimon-hive/paimon-hive-catalog/src/main/java/org/apache/paimon/hive/HiveCatalogOptions.java
+++
b/paimon-hive/paimon-hive-catalog/src/main/java/org/apache/paimon/hive/HiveCatalogOptions.java
@@ -48,6 +48,15 @@ public final class HiveCatalogOptions {
+ "If not configured, try to load from
'HADOOP_CONF_DIR' or 'HADOOP_HOME' system environment.\n"
+ "Configure Priority: 1.from
'hadoop-conf-dir' 2.from HADOOP_CONF_DIR 3.from HADOOP_HOME/conf
4.HADOOP_HOME/etc/hadoop.\n");
+ public static final ConfigOption<String> METASTORE_CLIENT_CLASS =
+ ConfigOptions.key("metastore.client.class")
+ .stringType()
+
.defaultValue("org.apache.hadoop.hive.metastore.HiveMetaStoreClient")
+ .withDescription(
+ "Class name of Hive metastore client.\n"
+ + "NOTE: This class must directly
implements "
+ +
"org.apache.hadoop.hive.metastore.IMetaStoreClient.");
+
public static final ConfigOption<Boolean> LOCATION_IN_PROPERTIES =
ConfigOptions.key("location-in-properties")
.booleanType()
