(polaris-tools) branch main updated: Fix and enable Hive to polaris migration test (#35)

[ajantha]

This is an automated email from the ASF dual-hosted git repository.

ajantha pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/polaris-tools.git


The following commit(s) were added to refs/heads/main by this push:
     new e61d81b  Fix and enable Hive to polaris migration test (#35)
e61d81b is described below

commit e61d81ba7ef485b8ad054a78907b0df2ed30ec45
Author: Ajantha Bhat <ajanthab...@gmail.com>
AuthorDate: Thu Oct 9 11:56:27 2025 +0530

    Fix and enable Hive to polaris migration test (#35)
---
 iceberg-catalog-migrator/README.md                                    | 4 ++++
 .../iceberg/catalog/migrator/cli/ITHiveToPolarisCLIMigrationTest.java | 2 --
 .../apache/polaris/iceberg/catalog/migrator/cli/PolarisContainer.java | 1 +
 3 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/iceberg-catalog-migrator/README.md 
b/iceberg-catalog-migrator/README.md
index b0ac4b5..4128f4d 100644
--- a/iceberg-catalog-migrator/README.md
+++ b/iceberg-catalog-migrator/README.md
@@ -218,6 +218,10 @@ java -jar iceberg-catalog-migrator-cli-0.0.1.jar migrate \
 --target-catalog-properties 
uri=http://localhost:60904/api/catalog,warehouse=test,token=$TOKEN
 ```
 
+Note: Need to configure `ALLOW_UNSTRUCTURED_TABLE_LOCATION` property at the 
polaris server side as
+HMS creates a namespace folder with ".db" extension. Also need to configure 
`allowedLocations` to be 
+source catalog directory in `storage_configuration_info`.
+
 ## Migrate all tables from DYNAMODB catalog to Polaris catalog
 ```shell
 java -jar iceberg-catalog-migrator-cli-0.0.1.jar migrate \
diff --git 
a/iceberg-catalog-migrator/cli/src/test/java/org/apache/polaris/iceberg/catalog/migrator/cli/ITHiveToPolarisCLIMigrationTest.java
 
b/iceberg-catalog-migrator/cli/src/test/java/org/apache/polaris/iceberg/catalog/migrator/cli/ITHiveToPolarisCLIMigrationTest.java
index d1e8621..95800c7 100644
--- 
a/iceberg-catalog-migrator/cli/src/test/java/org/apache/polaris/iceberg/catalog/migrator/cli/ITHiveToPolarisCLIMigrationTest.java
+++ 
b/iceberg-catalog-migrator/cli/src/test/java/org/apache/polaris/iceberg/catalog/migrator/cli/ITHiveToPolarisCLIMigrationTest.java
@@ -26,10 +26,8 @@ import org.apache.iceberg.hive.HiveMetastoreExtension;
 import org.apache.polaris.iceberg.catalog.migrator.api.CatalogMigrationUtil;
 import org.junit.jupiter.api.AfterAll;
 import org.junit.jupiter.api.BeforeAll;
-import org.junit.jupiter.api.Disabled;
 import org.junit.jupiter.api.extension.RegisterExtension;
 
-@Disabled("Because of https://github.com/apache/polaris/issues/2756";)
 public class ITHiveToPolarisCLIMigrationTest extends AbstractCLIMigrationTest {
 
   @RegisterExtension
diff --git 
a/iceberg-catalog-migrator/cli/src/test/java/org/apache/polaris/iceberg/catalog/migrator/cli/PolarisContainer.java
 
b/iceberg-catalog-migrator/cli/src/test/java/org/apache/polaris/iceberg/catalog/migrator/cli/PolarisContainer.java
index 699e82d..229316e 100644
--- 
a/iceberg-catalog-migrator/cli/src/test/java/org/apache/polaris/iceberg/catalog/migrator/cli/PolarisContainer.java
+++ 
b/iceberg-catalog-migrator/cli/src/test/java/org/apache/polaris/iceberg/catalog/migrator/cli/PolarisContainer.java
@@ -68,6 +68,7 @@ public class PolarisContainer extends 
GenericContainer<PolarisContainer> {
     jvmOptions.put("polaris.readiness.ignore-severe-issues", "true");
     jvmOptions.put("polaris.features.\"SUPPORTED_CATALOG_STORAGE_TYPES\"", 
"[\"FILE\"]");
     jvmOptions.put("polaris.features.\"ALLOW_INSECURE_STORAGE_TYPES\"", 
"true");
+    jvmOptions.put("polaris.features.\"ALLOW_UNSTRUCTURED_TABLE_LOCATION\"", 
"true");
     String jvmOptionsString =
         jvmOptions.entrySet().stream()
             .map(e -> "-D" + e.getKey() + "=" + e.getValue())