This is an automated email from the ASF dual-hosted git repository.
adutra pushed a commit to branch feature/iceberg-1.11
in repository https://gitbox.apache.org/repos/asf/polaris.git
The following commit(s) were added to refs/heads/feature/iceberg-1.11 by this
push:
new 4e2a5141f [Iceberg 1.11] Temporarily disable incompatible tests (#3684)
4e2a5141f is described below
commit 4e2a5141fc88aac9e8dce834c009b4bb6724f23e
Author: Alexandre Dutra <[email protected]>
AuthorDate: Fri Feb 6 16:16:57 2026 +0100
[Iceberg 1.11] Temporarily disable incompatible tests (#3684)
`CatalogTests.testLoadTableWithMissingMetadataFile` is not compatible with
REST catalogs, see https://github.com/apache/iceberg/pull/15192. It should
never be re-enabled.
The register view tests are failing for now since the register view
endpoint is not implemented. These tests should be re-enabled later.
---
.../it/test/PolarisApplicationIntegrationTest.java | 3 +--
.../it/test/PolarisRestCatalogIntegrationBase.java | 8 ++++++++
.../it/test/PolarisRestCatalogViewIntegrationBase.java | 16 ++++++++++++++++
.../catalog/iceberg/AbstractIcebergCatalogTest.java | 8 ++++++++
.../catalog/iceberg/AbstractIcebergCatalogViewTest.java | 16 ++++++++++++++++
5 files changed, 49 insertions(+), 2 deletions(-)
diff --git
a/integration-tests/src/main/java/org/apache/polaris/service/it/test/PolarisApplicationIntegrationTest.java
b/integration-tests/src/main/java/org/apache/polaris/service/it/test/PolarisApplicationIntegrationTest.java
index 5b53e80e2..4a066869c 100644
---
a/integration-tests/src/main/java/org/apache/polaris/service/it/test/PolarisApplicationIntegrationTest.java
+++
b/integration-tests/src/main/java/org/apache/polaris/service/it/test/PolarisApplicationIntegrationTest.java
@@ -281,8 +281,7 @@ public class PolarisApplicationIntegrationTest {
public void testConfigureCatalogCaseSensitive() {
assertThatThrownBy(() ->
newSessionCatalog("TESTCONFIGURECATALOGCASESENSITIVE"))
.isInstanceOf(RESTException.class)
- .hasMessage(
- "Unable to process: Unable to find warehouse
TESTCONFIGURECATALOGCASESENSITIVE");
+ .hasMessageContaining("Unable to find warehouse
TESTCONFIGURECATALOGCASESENSITIVE");
}
@Test
diff --git
a/integration-tests/src/main/java/org/apache/polaris/service/it/test/PolarisRestCatalogIntegrationBase.java
b/integration-tests/src/main/java/org/apache/polaris/service/it/test/PolarisRestCatalogIntegrationBase.java
index 69a39c7b0..9b1a96d87 100644
---
a/integration-tests/src/main/java/org/apache/polaris/service/it/test/PolarisRestCatalogIntegrationBase.java
+++
b/integration-tests/src/main/java/org/apache/polaris/service/it/test/PolarisRestCatalogIntegrationBase.java
@@ -34,6 +34,7 @@ import jakarta.ws.rs.core.Response;
import java.io.IOException;
import java.lang.reflect.Method;
import java.net.URI;
+import java.nio.file.Path;
import java.util.Arrays;
import java.util.HashMap;
import java.util.List;
@@ -118,9 +119,11 @@ import org.junit.jupiter.api.AfterAll;
import org.junit.jupiter.api.AfterEach;
import org.junit.jupiter.api.BeforeAll;
import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Disabled;
import org.junit.jupiter.api.Test;
import org.junit.jupiter.api.TestInfo;
import org.junit.jupiter.api.extension.ExtendWith;
+import org.junit.jupiter.api.io.TempDir;
/**
* Import the full core Iceberg catalog tests by hitting the REST service via
the RESTCatalog
@@ -2299,4 +2302,9 @@ public abstract class PolarisRestCatalogIntegrationBase
extends CatalogTests<RES
assertThat(nsResponse.namespaces()).hasSize(5);
assertThat(nsResponse.nextPageToken()).isNull();
}
+
+ @Test
+ @Disabled("Test is not compatible with REST catalogs")
+ @Override
+ public void testLoadTableWithMissingMetadataFile(@TempDir Path tempDir) {}
}
diff --git
a/integration-tests/src/main/java/org/apache/polaris/service/it/test/PolarisRestCatalogViewIntegrationBase.java
b/integration-tests/src/main/java/org/apache/polaris/service/it/test/PolarisRestCatalogViewIntegrationBase.java
index 778ea30a5..a12e2a866 100644
---
a/integration-tests/src/main/java/org/apache/polaris/service/it/test/PolarisRestCatalogViewIntegrationBase.java
+++
b/integration-tests/src/main/java/org/apache/polaris/service/it/test/PolarisRestCatalogViewIntegrationBase.java
@@ -51,6 +51,7 @@ import org.junit.jupiter.api.AfterAll;
import org.junit.jupiter.api.AfterEach;
import org.junit.jupiter.api.BeforeAll;
import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Disabled;
import org.junit.jupiter.api.Test;
import org.junit.jupiter.api.TestInfo;
import org.junit.jupiter.api.extension.ExtendWith;
@@ -236,4 +237,19 @@ public abstract class
PolarisRestCatalogViewIntegrationBase extends ViewCatalogT
.isNotNull()
.startsWith(customLocation);
}
+
+ @Test
+ @Disabled("Polaris REST catalog does not support view registration")
+ @Override
+ public void registerView() {}
+
+ @Test
+ @Disabled("Polaris REST catalog does not support view registration")
+ @Override
+ public void registerExistingView() {}
+
+ @Test
+ @Disabled("Polaris REST catalog does not support view registration")
+ @Override
+ public void registerViewThatAlreadyExistsAsTable() {}
}
diff --git
a/runtime/service/src/test/java/org/apache/polaris/service/catalog/iceberg/AbstractIcebergCatalogTest.java
b/runtime/service/src/test/java/org/apache/polaris/service/catalog/iceberg/AbstractIcebergCatalogTest.java
index 2f4fbfd18..5fbdbf029 100644
---
a/runtime/service/src/test/java/org/apache/polaris/service/catalog/iceberg/AbstractIcebergCatalogTest.java
+++
b/runtime/service/src/test/java/org/apache/polaris/service/catalog/iceberg/AbstractIcebergCatalogTest.java
@@ -41,6 +41,7 @@ import jakarta.inject.Inject;
import java.io.IOException;
import java.io.UncheckedIOException;
import java.lang.reflect.Method;
+import java.nio.file.Path;
import java.time.Clock;
import java.util.Arrays;
import java.util.Comparator;
@@ -165,8 +166,10 @@ import org.junit.jupiter.api.AfterEach;
import org.junit.jupiter.api.Assumptions;
import org.junit.jupiter.api.BeforeAll;
import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Disabled;
import org.junit.jupiter.api.Test;
import org.junit.jupiter.api.TestInfo;
+import org.junit.jupiter.api.io.TempDir;
import org.junit.jupiter.params.ParameterizedTest;
import org.junit.jupiter.params.provider.Arguments;
import org.junit.jupiter.params.provider.MethodSource;
@@ -2503,4 +2506,9 @@ public abstract class AbstractIcebergCatalogTest extends
CatalogTests<IcebergCat
}
}
}
+
+ @Test
+ @Disabled("Test is not compatible with REST catalogs")
+ @Override
+ public void testLoadTableWithMissingMetadataFile(@TempDir Path tempDir) {}
}
diff --git
a/runtime/service/src/test/java/org/apache/polaris/service/catalog/iceberg/AbstractIcebergCatalogViewTest.java
b/runtime/service/src/test/java/org/apache/polaris/service/catalog/iceberg/AbstractIcebergCatalogViewTest.java
index 0648ed343..a6e7d29c8 100644
---
a/runtime/service/src/test/java/org/apache/polaris/service/catalog/iceberg/AbstractIcebergCatalogViewTest.java
+++
b/runtime/service/src/test/java/org/apache/polaris/service/catalog/iceberg/AbstractIcebergCatalogViewTest.java
@@ -72,6 +72,7 @@ import
org.assertj.core.configuration.PreferredAssumptionException;
import org.junit.jupiter.api.AfterEach;
import org.junit.jupiter.api.BeforeAll;
import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Disabled;
import org.junit.jupiter.api.Test;
import org.junit.jupiter.api.TestInfo;
import org.junit.jupiter.api.io.TempDir;
@@ -268,4 +269,19 @@ public abstract class AbstractIcebergCatalogViewTest
extends ViewCatalogTests<Ic
afterRefreshEvent.attributes().getRequired(EventAttributes.VIEW_IDENTIFIER))
.isEqualTo(TestData.TABLE);
}
+
+ @Test
+ @Disabled("Polaris REST catalog does not support view registration")
+ @Override
+ public void registerView() {}
+
+ @Test
+ @Disabled("Polaris REST catalog does not support view registration")
+ @Override
+ public void registerExistingView() {}
+
+ @Test
+ @Disabled("Polaris REST catalog does not support view registration")
+ @Override
+ public void registerViewThatAlreadyExistsAsTable() {}
}
