Skip to content

Refactor HiveServer2's nativeTest#37401

Merged
sandynz merged 1 commit into
apache:masterfrom
linghengqian:dep-bump
Dec 16, 2025
Merged

Refactor HiveServer2's nativeTest#37401
sandynz merged 1 commit into
apache:masterfrom
linghengqian:dep-bump

Conversation

@linghengqian

@linghengqian linghengqian commented Dec 16, 2025

Copy link
Copy Markdown
Member

For #29052 .

Changes proposed in this pull request:


Before committing this PR, I'm sure that I have checked the following options:

  • My code follows the code of conduct of this project.
  • I have self-reviewed the commit code.
  • I have (or in comment I request) added corresponding labels for the pull request.
  • I have passed maven check locally : ./mvnw clean install -B -T1C -Dmaven.javadoc.skip -Dmaven.jacoco.skip -e.
  • I have made corresponding changes to the documentation.
  • I have added corresponding unit tests for my changes.
  • I have updated the Release Notes of the current development version. For more details, see Update Release Note

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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.

@sandynz
sandynz merged commit b54c814 into apache:master Dec 16, 2025
148 checks passed
@linghengqian
linghengqian deleted the dep-bump branch December 16, 2025 12:06
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)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants