Skip to content

HIVE-29485: Upgrade Ant version to 1.10.15#6348

Merged
ayushtkn merged 1 commit intoapache:masterfrom
hemanthumashankar0511:ant-upgrade
Mar 9, 2026
Merged

HIVE-29485: Upgrade Ant version to 1.10.15#6348
ayushtkn merged 1 commit intoapache:masterfrom
hemanthumashankar0511:ant-upgrade

Conversation

@hemanthumashankar0511
Copy link
Contributor

What changes were proposed in this pull request?
This PR upgrades the Apache Ant version from 1.10.13 to 1.10.15. The change is implemented by updating the <ant.version> property in the root pom.xml, which centralizes the version management for all child modules including ql, common, and vector-code-gen.

Why are the changes needed?
The upgrade is necessary to improve compatibility with modern Java environments and address internal build-tool bugs:

Java 22+ Support: Ant 1.10.15 officially adds support for Java 22 and higher.

JDK Compatibility: It fixes an issue (Bugzilla 68460) where Ant referenced internal JDK classes that have been removed in newer Java versions, which could lead to build failures in modern environments.

Security & Stability: It includes important fixes for the junitlauncher task and security-related improvements for temporary file handling introduced in the 1.10.x line.

Does this PR introduce any user-facing change?
No. This is strictly a change to the build-time dependency and does not affect Hive's runtime APIs or SQL features.

How was this patch tested?
The following verification steps were performed locally to ensure no regressions in code generation or build stability:

Property Resolution: Confirmed that ${ant.version} correctly resolves to 1.10.15 using mvn help:evaluate -Dexpression=ant.version.

Dependency Tree: Verified that no older versions of Ant are being pulled in transitively using mvn dependency:tree -Dincludes=org.apache.ant:ant.

Code Generation Check: Successfully built modules that rely on Ant for code and grammar generation: mvn clean install -pl ql,vector-code-gen -DskipTests.

Unit Test Sanity: Ran parser tests in the ql module to ensure the generated grammar remains functional: mvn test -pl ql -Dtest=TestParse.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 6, 2026

Copy link
Member

@ayushtkn ayushtkn left a comment

Choose a reason for hiding this comment

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

Changes LGTM.

@hemanthumashankar0511 can you check the mvn dependency:tree to see there aren't multiple version on ant coming & we are in sync

@hemanthumashankar0511
Copy link
Contributor Author

Changes LGTM.

@hemanthumashankar0511 can you check the mvn dependency:tree to see there aren't multiple version on ant coming & we are in sync

I checked the dependency tree for any conflicts....
org.apache.ant:ant:1.10.15 is the only version coming in, consistently across all 56 modules. No version conflicts.

@ayushtkn ayushtkn merged commit f301a6c into apache:master Mar 9, 2026
2 checks passed
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