Skip to content

Commit d9acb4f

Browse files
committed
test new rest catalog image
1 parent 68e17af commit d9acb4f

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

dev/docker-compose-integration.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,8 @@ services:
4141
- hive:hive
4242
- minio:minio
4343
rest:
44-
image: tabulario/iceberg-rest
44+
# TODO: This needs to be updated to apache/iceberg-rest-fixture once https://github.com/apache/iceberg/pull/11676 is merged
45+
image: asfjenkins/iceberg-rest-fixture
4546
container_name: pyiceberg-rest
4647
networks:
4748
iceberg_net:

tests/integration/test_writes/test_writes.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1368,10 +1368,9 @@ def test_table_v1_with_null_nested_namespace(session_catalog: Catalog, arrow_tab
13681368
identifier = "default.lower.table_v1_with_null_nested_namespace"
13691369
tbl = _create_table(session_catalog, identifier, {"format-version": "1"}, [arrow_table_with_null])
13701370
assert tbl.format_version == 1, f"Expected v1, got: v{tbl.format_version}"
1371-
# TODO: Add session_catalog.table_exists check here when we integrate a REST catalog image
1372-
# that supports HEAD request on table endpoint
13731371

1374-
# assert session_catalog.table_exists(identifier)
1372+
assert session_catalog.load_table(identifier) is not None
1373+
assert session_catalog.table_exists(identifier)
13751374

13761375
# We expect no error here
13771376
session_catalog.drop_table(identifier)

0 commit comments

Comments
 (0)