Skip to content

Fix release build: upgrade maven-compiler-plugin and consolidate Java compiler release target#802

Open
jordanpadams wants to merge 2 commits into
developfrom
fix/maven-compiler-plugin-release
Open

Fix release build: upgrade maven-compiler-plugin and consolidate Java compiler release target#802
jordanpadams wants to merge 2 commits into
developfrom
fix/maven-compiler-plugin-release

Conversation

@jordanpadams

@jordanpadams jordanpadams commented Jul 17, 2026

Copy link
Copy Markdown
Member

🗒️ Summary

Fixes a release build failure caused by maven-compiler-plugin:3.8.1 not supporting Java release target 21. The service module overrode maven.compiler.release to 21, but the old plugin version predates Java 21 support, producing: Fatal error compiling: error: release version 21 not supported.

Changes:

  • Upgrade maven-compiler-plugin from 3.8.13.13.0 in root pom.xml (adds Java 21 support)
  • Remove maven.compiler.release=21 override from service/pom.xml — no Java 21 language features are in use; service runs on JVM 21 at runtime via BP_JVM_VERSION
  • Remove redundant maven.compiler.release=17 from model/pom.xml — already the root default
  • Remove unused maven-compiler-plugin.version property from lexer/pom.xml

All submodules now inherit maven.compiler.release=17 from the root pom as a single source of truth.

🤖 AI Assistance Disclosure

  • No AI assistance used
  • AI used for light assistance (e.g., suggestions, refactoring, documentation help, minor edits)
  • AI used for moderate content generation (AI generated some code or logic, but the developer authored or heavily revised the majority)
  • AI generated substantial portions of this code

Estimated % of code influenced by AI: 75%

⚙️ Test Data and/or Report

Build configuration fix only — no application logic changed. Validation is that the stable release build which previously failed with release version 21 not supported should now succeed.

Note: Integration tests are failing, but not sure there is much we can do about that right now with the repos a bit out of sync with backwards-incompatible changes

♻️ Related Issues

N/A — fixing stable release pipeline regression.

🤓 Reviewer Checklist

Reviewers: Please verify the following before approving this pull request.

Documentation and PR Content

  • Documentation: README, Wiki, or inline documentation (Sphinx, Javadoc, Docstrings) have been updated to reflect these changes.
  • Issue Traceability: The PR is linked to a valid GitHub Issue
  • PR Title: The PR title is "user-friendly" clearly identifying what is being fixed or the new feature being added, that if you saw it in the Release Notes for a tool, you would be able to get the gist of what was done.

Security & Quality

  • SonarCloud: Confirmed no new High or Critical security findings.
  • Secrets Detection: Verified that the Secrets Detection scan passed and no sensitive information (keys, tokens, PII) is exposed.
  • Code Quality: Code follows organization style guidelines and best practices for the specific language (e.g., PEP 8, Google Java Style).

Testing & Validation

  • Test Accuracy: Verified that test data is accurate, representative of real-world PDS4 scenarios, and sufficient for the logic being tested.
  • Coverage: Automated tests cover new logic and edge cases.
  • Local Verification: (If applicable) Successfully built and ran the changes in a local or staging environment.

Maintenance

  • Backward Compatibility: Confirmed that these changes do not break existing downstream dependencies or API contracts (or that breaking changes are clearly documented).

Upgrade maven-compiler-plugin from 3.8.1 to 3.13.0 in root pom to
support Java 21 release target. Remove per-submodule maven.compiler.release
overrides so all modules inherit the single source-of-truth (release 17)
from the root pom, eliminating the divergence that broke the stable release build.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@jordanpadams
jordanpadams requested a review from a team as a code owner July 17, 2026 16:07
@jordanpadams jordanpadams added the bug Something isn't working label Jul 17, 2026
@jordanpadams jordanpadams self-assigned this Jul 17, 2026
@jordanpadams jordanpadams added the bug Something isn't working label Jul 17, 2026
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant