You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@age.apache.org by "jrgemignani (via GitHub)" <gi...@apache.org> on 2026/02/13 00:27:17 UTC

[PR] Fix JDBC driver CI test failures: encoding and Testcontainers [age]

jrgemignani opened a new pull request, #2333:
URL: https://github.com/apache/age/pull/2333

   Note: The PR was created with the help of AI tools and a human.
   
   Fix two root causes of CI test failures reproduced in ubuntu:24.04 container with Zulu Java 17 + Gradle 9.3.1 (matching GitHub Actions):
   
   build.gradle.kts: Set javac encoding to UTF-8 to prevent Unicode test failures (e.g. Greek Omega) when the CI runner locale defaults to US-ASCII. Without this, source characters are garbled at compile time, causing deterministic assertion failures in AgtypeFactoryTest and AgtypeUtilTest.
   
   BaseDockerizedTest.java: Fix Testcontainers setup reliability -
   - Add Wait.forLogMessage wait strategy with 60s timeout so PostgreSQL is fully ready before JDBC connection attempt
   - Use getHost() instead of hardcoded "localhost" for correct behavior in Docker-in-Docker and varied CI environments
   - Add sslmode=disable to JDBC URL since PostgreSQL driver 42.6.0+ attempts SSL negotiation by default and the AGE Docker image has no SSL configured
   - Remove silent catch/println that swallowed connection errors and produced misleading NullPointerExceptions downstream
   
   Verified: 77/77 tests pass across multiple consecutive runs.
   
   modified:   drivers/jdbc/lib/build.gradle.kts
   modified:   drivers/jdbc/lib/src/test/java/org/apache/age/jdbc/BaseDockerizedTest.java


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@age.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Fix JDBC driver CI test failures: encoding and Testcontainers [age]

Posted by "MuhammadTahaNaveed (via GitHub)" <gi...@apache.org>.
MuhammadTahaNaveed merged PR #2333:
URL: https://github.com/apache/age/pull/2333


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@age.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org