Refactor HiveServer2's nativeTest#37401
Merged
Merged
Conversation
linghengqian
force-pushed
the
dep-bump
branch
from
December 16, 2025 07:40
4f9faca to
557ba84
Compare
linghengqian
marked this pull request as ready for review
December 16, 2025 08:57
There was a problem hiding this comment.
Pull request overview
This PR refactors HiveServer2's native tests to consolidate Iceberg table creation logic and updates multiple dependencies and Docker images used in native tests.
Key changes:
- Bumps GraalVM Native Build Tools from 0.11.0 to 0.11.3, removing the workaround for a previously unresolved issue
- Updates Docker images for PostgreSQL (17.5 → 18.1), MySQL (9.4.0 → 9.5.0), SQL Server (2022-CU20 → 2025-RTM), and Presto (0.292 → 0.296)
- Refactors HiveServer2 test code to centralize Iceberg table creation in repository classes instead of duplicating logic across test files
- Fixes incorrect host paths in Seata documentation
Reviewed changes
Copilot reviewed 26 out of 26 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| pom.xml | Bumps GraalVM Native Build Tools to 0.11.3, JUnit to 5.14.1, Presto to 0.296, and removes manual metadata-copy workaround |
| test/native/src/test/resources/test-native/yaml/jdbc/databases/postgresql.yaml | Updates PostgreSQL Docker image to 18.1-trixie |
| test/native/src/test/resources/test-native/yaml/jdbc/databases/sqlserver.yaml | Updates SQL Server Docker image to 2025-RTM-ubuntu-22.04 |
| test/native/src/test/resources/test-native/yaml/jdbc/transactions/base/seata.yaml | Updates PostgreSQL to 18.1 and fixes JDBC URL host paths |
| test/native/src/test/resources/container-license-acceptance.txt | Updates SQL Server container license acceptance |
| test/native/src/test/java/org/apache/shardingsphere/test/natived/proxy/databases/PostgresTest.java | Updates PostgreSQL container to 18.1-trixie |
| test/native/src/test/java/org/apache/shardingsphere/test/natived/proxy/databases/MySQLTest.java | Updates MySQL container to 9.5.0-oraclelinux9 |
| test/native/src/test/java/org/apache/shardingsphere/test/natived/proxy/transactions/base/SeataTest.java | Updates PostgreSQL container to 18.1-trixie |
| test/native/src/test/java/org/apache/shardingsphere/test/natived/jdbc/databases/SQLServerTest.java | Adds logging configuration to suppress verbose SQL Server connection warnings |
| test/native/src/test/java/org/apache/shardingsphere/test/natived/jdbc/databases/PrestoTest.java | Updates Presto container to 0.296 |
| test/native/src/test/java/org/apache/shardingsphere/test/natived/jdbc/databases/MySQLTest.java | Updates MySQL container to 9.5.0-oraclelinux9 |
| test/native/src/test/java/org/apache/shardingsphere/test/natived/jdbc/databases/hive/ZookeeperServiceDiscoveryTest.java | Refactors test to use centralized table creation, renames methods for clarity, removes unused import |
| test/native/src/test/java/org/apache/shardingsphere/test/natived/jdbc/databases/hive/SystemSchemasTest.java | Refactors test to use centralized table creation, updates PostgreSQL to 18.1, removes unused import and initTable method |
| test/native/src/test/java/org/apache/shardingsphere/test/natived/jdbc/databases/hive/IcebergTest.java | Refactors test to use centralized table creation, removes unused import and initTable method |
| test/native/src/test/java/org/apache/shardingsphere/test/natived/commons/repository/OrderRepository.java | Adds createIcebergTableInHiveServer2 method for centralized table creation |
| test/native/src/test/java/org/apache/shardingsphere/test/natived/commons/repository/OrderItemRepository.java | Adds createIcebergTableInHiveServer2 method for centralized table creation |
| test/native/src/test/java/org/apache/shardingsphere/test/natived/commons/repository/AddressRepository.java | Adds createIcebergTableInHiveServer2 method for centralized table creation |
| infra/reachability-metadata/src/main/resources/META-INF/native-image/org.apache.shardingsphere/shardingsphere-infra-reachability-metadata/reachability-metadata.json | Adds GraalVM reachability metadata for YamlShardingSphereIndexBeanInfo and YamlShardingSphereIndexCustomizer |
| docs/document/content/user-manual/shardingsphere-jdbc/special-api/transaction/seata.en.md | Fixes incorrect Docker volume host path |
| docs/document/content/user-manual/shardingsphere-jdbc/special-api/transaction/seata.cn.md | Fixes incorrect Docker volume host path |
| docs/document/content/user-manual/shardingsphere-jdbc/optional-plugins/presto/_index.en.md | Updates Presto version to 0.296 in documentation and examples |
| docs/document/content/user-manual/shardingsphere-jdbc/optional-plugins/presto/_index.cn.md | Updates Presto version to 0.296 in documentation and examples |
| docs/document/content/user-manual/shardingsphere-jdbc/optional-plugins/hiveserver2/_index.en.md | Updates hive-server2-jdbc-driver-thin version to 1.8.2, removes manual table creation instructions, adds table creation to code examples |
| docs/document/content/user-manual/shardingsphere-jdbc/optional-plugins/hiveserver2/_index.cn.md | Updates hive-server2-jdbc-driver-thin version to 1.8.2, removes manual table creation instructions, adds table creation to code examples |
| docs/document/content/user-manual/shardingsphere-jdbc/graalvm-native-image/_index.en.md | Updates GraalVM Native Build Tools version to 0.11.3 |
| docs/document/content/user-manual/shardingsphere-jdbc/graalvm-native-image/_index.cn.md | Updates GraalVM Native Build Tools version to 0.11.3 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
linghengqian
force-pushed
the
dep-bump
branch
from
December 16, 2025 10:55
557ba84 to
d64386c
Compare
sandynz
approved these changes
Dec 16, 2025
makssent
pushed a commit
to makssent/shardingsphere
that referenced
this pull request
Apr 9, 2026
…ngsphere/master * remotes/origin/master: Refactor HiveServer2's nativeTest (apache#37401)
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.
For #29052 .
Changes proposed in this pull request:
metadata-copyMaven goal to simplify the use of integration tests graalvm/native-build-tools#748 . This PR was largely split from Support building Proxy Native via GraalVM CE for JDK 25 #37357, which has some unaddressed issues with GraalVM JDK for JDK 25.mysql,postgres,mcr.microsoft.com/mssql/server, andprestodb/presto. Also bump the Presto JDBC driver to a matching version.opengauss/opengauss-serverdid not upload the7.0.0-RC2Docker image to Docker Hub.clickhouse/clickhouse-serverhas been broken, which requires a separate issue for discussion. See Version 25.10 breaks JBDC driver ClickHouse/clickhouse-java#2636 .dyrnq/doris:4.0.0cannot start normally likedyrnq/doris:3.1.0dyrnq/docker-doris#38,dyrnq/doriscannot be updated to4.0.0.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.