Skip to content

Conversation

@Jefffrey
Copy link
Contributor

@Jefffrey Jefffrey commented Jan 3, 2026

Which issue does this PR close?

Rationale for this change

We don't actually need the dependency on testcontainers per their README:

Note: you don't need to explicitly depend on testcontainers as it's re-exported dependency of testcontainers-modules with aligned version between these crates.

It was causing some version conflict issues when we tried to bump testcontainers-modules so remove it and keep only modules, fixing the code to use the re-exports.

What changes are included in this PR?

Remove testcontainers dependency, amend imports to use re-exports of testcontainers-modules.

Bump testcontainers-modules from 0.13 to 0.14.

Are these changes tested?

CLI test suite compiles and runs. Ran the SLT postgres compat mode locally successfully:

datafusion (testcontainers-0.14)$ PG_COMPAT=true PG_URI="postgresql://[email protected]/postgres" cargo test --features=postgres --test sqllogictests
    Finished `test` profile [unoptimized + debuginfo] target(s) in 0.19s
     Running bin/sqllogictests.rs (/Users/jeffrey/.cargo_target_cache/debug/deps/sqllogictests-223c346f75c524e2)
Completed 6 test files in 0 seconds 

Are there any user-facing changes?

No.

Copy link
Contributor

@comphead comphead left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @Jefffrey

@comphead comphead added this pull request to the merge queue Jan 3, 2026
Merged via the queue into apache:main with commit 09455f1 Jan 3, 2026
31 checks passed
@Jefffrey Jefffrey deleted the testcontainers-0.14 branch January 4, 2026 03:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sqllogictest SQL Logic Tests (.slt)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants