Fix nativeTest under GraalVM Native Image with clickhouse-server 25.12#37440
Merged
terrymanu merged 1 commit intoapache:masterfrom Dec 20, 2025
Merged
Fix nativeTest under GraalVM Native Image with clickhouse-server 25.12#37440terrymanu merged 1 commit intoapache:masterfrom
25.12#37440terrymanu merged 1 commit intoapache:masterfrom
Conversation
There was a problem hiding this comment.
Pull request overview
This PR fixes native test execution under GraalVM Native Image by upgrading to ClickHouse server version 25.12, and addresses edge cases with GraalVM CE for JDK 24 by bumping several dependencies and the Maven wrapper.
- Updates ClickHouse server from 25.10.3.100 to 25.12.1.649 and ClickHouse JDBC driver from 0.9.4 to 0.9.5
- Updates Lombok from 1.18.38 to 1.18.42 and Firebird JDBC driver (Jaybird) from 5.0.6.java8 to 5.0.10.java8
- Upgrades Maven wrapper from 3.3.2 to 3.3.4 and Maven from 3.9.11 to 3.9.12
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| pom.xml | Updates version properties for Lombok (1.18.42), ClickHouse JDBC driver (0.9.5), and Jaybird (5.0.10.java8) |
| test/native/src/test/resources/test-native/yaml/jdbc/databases/clickhouse.yaml | Updates ClickHouse server version in test JDBC URLs from 25.10.3.100 to 25.12.1.649 |
| mvnw | Updates Maven wrapper shell script to version 3.3.4 with improved directory handling logic |
| mvnw.cmd | Updates Maven wrapper batch script to version 3.3.4 with improved PowerShell directory handling |
| .mvn/wrapper/maven-wrapper.properties | Updates wrapper version to 3.3.4 and Maven distribution to 3.9.12 |
| infra/reachability-metadata/src/main/resources/META-INF/native-image/org.apache.shardingsphere/generated-reachability-metadata/reachability-metadata.json | Removes duplicate conditional registration for JDBCBackendDataSource |
| docs/document/content/user-manual/shardingsphere-jdbc/optional-plugins/clickhouse/_index.en.md | Updates ClickHouse JDBC driver version reference to 0.9.5 and server version to 25.12.1.649 |
| docs/document/content/user-manual/shardingsphere-jdbc/optional-plugins/clickhouse/_index.cn.md | Updates ClickHouse JDBC driver version reference to 0.9.5 and server version to 25.12.1.649 (Chinese documentation) |
| docs/document/content/user-manual/shardingsphere-jdbc/optional-plugins/firebird/_index.en.md | Updates Jaybird version reference to 5.0.10.java8 |
| docs/document/content/user-manual/shardingsphere-jdbc/optional-plugins/firebird/_index.cn.md | Updates Jaybird version reference to 5.0.10.java8 (Chinese documentation) |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
terrymanu
approved these changes
Dec 20, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #37406 .
Changes proposed in this pull request:
25.12. This is the successor to Fix integration issues with clickhouse-server25.10#37421, except that it was previously waiting to be merged with Fix tests for new data type Geometry. issue is created and planned ClickHouse/clickhouse-java#2679 .Before committing this PR, I'm sure that I have checked the following options:
./mvnw clean install -B -T1C -Dmaven.javadoc.skip -Dmaven.jacoco.skip -e.