(hive) branch master updated: HIVE-29779: Revive some test cases in HiveIcebergRESTViewCatalogTests (#6660)

[dkuzmenko]

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

deniskuzZ pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hive.git


The following commit(s) were added to refs/heads/master by this push:
     new 23fe9939157 HIVE-29779: Revive some test cases in 
HiveIcebergRESTViewCatalogTests (#6660)
23fe9939157 is described below

commit 23fe9939157edc828a7ff8a59dc809f8c4eb894f
Author: Shohei Okumiya <oku...@apache.org>
AuthorDate: Sat Aug 8 19:33:05 2026 +0900

    HIVE-29779: Revive some test cases in HiveIcebergRESTViewCatalogTests 
(#6660)
---
 .../rest/HiveIcebergRESTViewCatalogTests.java      | 30 ++++++++--------------
 1 file changed, 10 insertions(+), 20 deletions(-)

diff --git 
a/itests/test-docker/src/test/java/org/apache/iceberg/rest/HiveIcebergRESTViewCatalogTests.java
 
b/itests/test-docker/src/test/java/org/apache/iceberg/rest/HiveIcebergRESTViewCatalogTests.java
index 20041bdbca9..41233c8cf6f 100644
--- 
a/itests/test-docker/src/test/java/org/apache/iceberg/rest/HiveIcebergRESTViewCatalogTests.java
+++ 
b/itests/test-docker/src/test/java/org/apache/iceberg/rest/HiveIcebergRESTViewCatalogTests.java
@@ -57,32 +57,22 @@ protected RESTCatalog tableCatalog() {
   }
 
   @Override
-  protected boolean requiresNamespaceCreate() {
-    return true;
-  }
-
-  @Override
-  protected boolean supportsServerSideRetry() {
-    return true;
-  }
+  protected String viewLocation(String... paths) {
+    StringBuilder location = new StringBuilder("s3a://test/test-warehouse");
+    for (String path : paths) {
+      location.append("/").append(path);
+    }
 
-  @Override
-  public void completeCreateView() {
-    // This test case requires https://github.com/apache/iceberg/pull/14653
-  }
-
-  @Override
-  public void createAndReplaceViewWithLocation() {
-    // This test case requires https://github.com/apache/iceberg/pull/14653
+    return location.toString();
   }
 
   @Override
-  public void createViewWithCustomMetadataLocation() {
-    // This test case requires https://github.com/apache/iceberg/pull/14653
+  protected boolean requiresNamespaceCreate() {
+    return true;
   }
 
   @Override
-  public void updateViewLocation() {
-    // This test case requires https://github.com/apache/iceberg/pull/14653
+  protected boolean supportsServerSideRetry() {
+    return true;
   }
 }