The GitHub Actions job "Python CI" on iceberg-python.git/test-trino-integration 
has failed.
Run started by GitHub user dingo4dev (triggered by dingo4dev).

Head commit for run:
774ec5feb71d475f446abdd14a6f841767d72da4 / Stanley Law 
<[email protected]>
Adds Docker Compose setup for Trino

Adds a Docker Compose configuration for setting up Trino with Iceberg, 
including support for Hive Metastore and REST catalog types. This allows for 
easier testing and development with Trino and Iceberg.

closes #2219

add uuid partitions test with trino

Add Trino as alternative tool for test uuid partitions as Java Iceberg 1.9.2 in 
spark is not yet supported.
<!--related to #2002-->

fix: correct conditions in namespace

fix: add license to trino config file

remove precommit as prek exist

use mark to skip BucketTransform in Spark uuid partition test

Trino: Restructure to make Trino integration optional and modular

Address reviewer feedback from kevinjqliu:

1. Consolidated Trino infrastructure:
   - All Trino config files remain in dev/trino/ directory
   - docker-compose-trino.yml moved to dev/ (alongside integration compose)
   - run-trino.sh moved to dev/ (alongside other run scripts)

2. Removed Trino from main integration docker-compose:
   - Trino service removed from dev/docker-compose-integration.yml
   - Trino can now be spun up separately alongside main integration
   - Keeps Trino testing optional and not part of CI

3. Created dedicated test file:
   - tests/integration/test_trino.py for all Trino-specific tests
   - Moved test_schema_exists_in_trino from test_rest_catalog.py
   - Moved test_uuid_partitioning_with_trino from test_writes.py
   - Better separation of concerns and easier to maintain

4. Simplified pytest marker:
   - Changed from @pytest.mark.integration_trino to @pytest.mark.trino
   - Updated Makefile target: test-integration-trino -> test-trino
   - Updated pyproject.toml and conftest.py references

This makes Trino integration testing opt-in and follows the same
pattern as other optional test suites (s3, adls, gcs).

Trino: Mount catalog files individually to preserve built-in catalogs

Address feedback from ebyhr on PR #2220 discussion r2583421945.

Instead of mounting the entire catalog directory, mount individual
catalog property files. This allows Trino to preserve its built-in
catalogs (memory, TPCH) which are helpful during development, while
still providing our custom Iceberg catalogs.

Mounted files:
- warehouse_rest.properties - REST catalog configuration
- warehouse_hive.properties - Hive catalog configuration
- config.properties - Trino server configuration

make lint

Report URL: https://github.com/apache/iceberg-python/actions/runs/21556478706

With regards,
GitHub Actions via GitBox

Reply via email to