From 89e2b73d26bbd9cb28094dd84953229a87716d82 Mon Sep 17 00:00:00 2001 From: Fokko Driesprong Date: Tue, 14 Apr 2026 22:25:24 +0200 Subject: [PATCH 1/2] CI: Lock REST Fixture to 1.10.1 With the new Jetty upgrade in https://github.com/apache/iceberg/pull/10837 we're running into the following: Failing CI: https://github.com/apache/iceberg-python/actions/runs/24416422109/job/71326697962 ```json Malformed request: { "message":"Suspicious Path Character", "url":"http://rest:8181/v1/namespaces/default%1Ftest_positional_mor_deletes_v2/tables/branch_without_5?snapshots=all", "status":"400" } ``` The `namespace-seperator` has been introduced afther the 1.10.x branch (https://github.com/apache/iceberg-python/pull/2826), so it isn't out in public, meaning the Spark integration tests are not respecting this config. This requires us to downgrade for now. I think disallowing the default (`%1F`) separator is problematic, since it might break compatibility. --- dev/docker-compose-integration.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dev/docker-compose-integration.yml b/dev/docker-compose-integration.yml index 03f5684ce4..387e54ac2c 100644 --- a/dev/docker-compose-integration.yml +++ b/dev/docker-compose-integration.yml @@ -44,7 +44,7 @@ services: retries: 5 start_period: 90s rest: - image: apache/iceberg-rest-fixture + image: apache/iceberg-rest-fixture:1.10.1 container_name: pyiceberg-rest networks: iceberg_net: From 14fa942bc93e1b4f5e2feaf63360bb67cf94d4e6 Mon Sep 17 00:00:00 2001 From: Fokko Driesprong Date: Wed, 15 Apr 2026 14:50:04 +0200 Subject: [PATCH 2/2] Add missing test file --- tests/integration/test_catalog.py | 1 + 1 file changed, 1 insertion(+) diff --git a/tests/integration/test_catalog.py b/tests/integration/test_catalog.py index 590c901914..751dbe0479 100644 --- a/tests/integration/test_catalog.py +++ b/tests/integration/test_catalog.py @@ -618,6 +618,7 @@ def test_register_table_existing(test_catalog: Catalog, table_schema_nested: Sch @pytest.mark.integration +@pytest.mark.skip(reason="Requires Iceberg REST Fixtures 1.11.x") def test_rest_custom_namespace_separator(rest_catalog: RestCatalog, table_schema_simple: Schema) -> None: """ Tests that the REST catalog correctly picks up the namespace-separator from the config endpoint.