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

mchades pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/gravitino.git


The following commit(s) were added to refs/heads/main by this push:
     new 3b08dd7a4 [#3783] improvement(test): add test 'list-all-tables' for 
the Hive catalog (#5710)
3b08dd7a4 is described below

commit 3b08dd7a446a6dbbcd98da374e192d050104ef45
Author: Yuan Chen <[email protected]>
AuthorDate: Sun Dec 1 21:04:39 2024 -0500

    [#3783] improvement(test): add test 'list-all-tables' for the Hive catalog 
(#5710)
    
    ### What changes were proposed in this pull request?
    
    add test 'list-all-tables'  for the Hive catalog
    
    ### Why are the changes needed?
    
    Fix: #3783
    
    ### Does this PR introduce _any_ user-facing change?
    
    N/A
    
    ### How was this patch tested?
    
     CI passed
---
 .../gravitino/catalog/hive/integration/test/CatalogHiveIT.java      | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git 
a/catalogs/catalog-hive/src/test/java/org/apache/gravitino/catalog/hive/integration/test/CatalogHiveIT.java
 
b/catalogs/catalog-hive/src/test/java/org/apache/gravitino/catalog/hive/integration/test/CatalogHiveIT.java
index 16ca3b57f..d9e6fe70d 100644
--- 
a/catalogs/catalog-hive/src/test/java/org/apache/gravitino/catalog/hive/integration/test/CatalogHiveIT.java
+++ 
b/catalogs/catalog-hive/src/test/java/org/apache/gravitino/catalog/hive/integration/test/CatalogHiveIT.java
@@ -620,6 +620,7 @@ public class CatalogHiveIT extends BaseIT {
     NameIdentifier icebergTable = NameIdentifier.of(schemaName, 
"iceberg_table");
     NameIdentifier paimonTable = NameIdentifier.of(schemaName, "paimon_table");
     NameIdentifier hudiTable = NameIdentifier.of(schemaName, "hudi_table");
+    NameIdentifier hiveTable = NameIdentifier.of(schemaName, "hive_table");
     catalog
         .asTableCatalog()
         .createTable(icebergTable, createColumns(), null, 
ImmutableMap.of("table_type", "ICEBERG"));
@@ -629,8 +630,11 @@ public class CatalogHiveIT extends BaseIT {
     catalog
         .asTableCatalog()
         .createTable(hudiTable, createColumns(), null, 
ImmutableMap.of("provider", "hudi"));
+    catalog
+        .asTableCatalog()
+        .createTable(hiveTable, createColumns(), null, 
ImmutableMap.of("provider", "hive"));
     NameIdentifier[] tables = 
catalog.asTableCatalog().listTables(Namespace.of(schemaName));
-    Assertions.assertEquals(0, tables.length);
+    Assertions.assertEquals(1, tables.length);
   }
 
   @Test

Reply via email to