testcontainers-java
testcontainers-java copied to clipboard
Change the Docker Image used by the Presto module to `prestodb/presto`
- Fixes #8657 .
- Change the Docker Image used by the Presto module to
prestodb/presto.
Thanks for your contribution, @linghengqian! In order to fix the build, please run ./gradlew :presto:spotlessApply.
Thanks for your contribution, @linghengqian! In order to fix the build, please run
./gradlew :presto:spotlessApply.
- To keep the original unit tests, I added a few Presto Connector files. There are some additional differences.
- Presto does not support calling
setAutoCommitrepeatedly on the same Connection. See https://github.com/prestodb/presto/issues/15916 . - Presto does not support the
ON TRUEcondition forCROSS JOIN UNNESTsyntax, and Presto does not support theLEFT JOIN UNNESTsyntax. See https://github.com/prestodb/presto/issues/8200 . - Presto does not support the
FETCH FIRST N ROWS WITH TIESsyntax. See https://github.com/prestodb/presto/issues/23223 . - I will say that I’m not really sure why unit tests often result in
Caused by: com.facebook.presto.spi.PrestoException: No nodes available to run query, as I haven’t been able to find a simple healthCheck method in the Presto documentation that can replace the following. Refer to https://github.com/prestodb/presto/issues/23226 .
this.waitStrategy =
new LogMessageWaitStrategy()
.withRegEx(".*======== SERVER STARTED ========.*")
.withStartupTimeout(Duration.of(60, ChronoUnit.SECONDS));
thanks for the update, @linghengqian ! I see everything green but failing locally. So, hopefully the other issue in presto repository is solved soon and the module can provide a reliable wait strategy.