From 9a9230e713595bc716cc1229dcb2e493a3cd30aa Mon Sep 17 00:00:00 2001 From: Roushan Kumar Date: Thu, 6 Feb 2025 15:12:31 +0530 Subject: [PATCH] rename getTableBuilder to getInstance to keep consistent with glue one --- .../main/java/org/apache/xtable/hms/HMSCatalogSyncClient.java | 3 +-- .../org/apache/xtable/hms/HMSCatalogTableBuilderFactory.java | 2 +- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/xtable-hive-metastore/src/main/java/org/apache/xtable/hms/HMSCatalogSyncClient.java b/xtable-hive-metastore/src/main/java/org/apache/xtable/hms/HMSCatalogSyncClient.java index db86958d9..537de8341 100644 --- a/xtable-hive-metastore/src/main/java/org/apache/xtable/hms/HMSCatalogSyncClient.java +++ b/xtable-hive-metastore/src/main/java/org/apache/xtable/hms/HMSCatalogSyncClient.java @@ -194,8 +194,7 @@ private void _init( } catch (MetaException | HiveException e) { throw new CatalogSyncException("HiveMetastoreClient could not be created", e); } - this.tableBuilder = - HMSCatalogTableBuilderFactory.getTableBuilder(tableFormat, this.configuration); + this.tableBuilder = HMSCatalogTableBuilderFactory.getInstance(tableFormat, this.configuration); } /** diff --git a/xtable-hive-metastore/src/main/java/org/apache/xtable/hms/HMSCatalogTableBuilderFactory.java b/xtable-hive-metastore/src/main/java/org/apache/xtable/hms/HMSCatalogTableBuilderFactory.java index 05cf2c29f..56c182798 100644 --- a/xtable-hive-metastore/src/main/java/org/apache/xtable/hms/HMSCatalogTableBuilderFactory.java +++ b/xtable-hive-metastore/src/main/java/org/apache/xtable/hms/HMSCatalogTableBuilderFactory.java @@ -40,7 +40,7 @@ public class HMSCatalogTableBuilderFactory { - public static CatalogTableBuilder getTableBuilder( + static CatalogTableBuilder getInstance( String tableFormat, Configuration configuration) { switch (tableFormat) { case TableFormat.ICEBERG: