[Improvement-18109][All Modules] Upgrade to Spring Boot 3.2#18110
Closed
singhularity wants to merge 1 commit intoapache:devfrom
Closed
[Improvement-18109][All Modules] Upgrade to Spring Boot 3.2#18110singhularity wants to merge 1 commit intoapache:devfrom
singhularity wants to merge 1 commit intoapache:devfrom
Conversation
Spring Boot 2.6.1 reached end of life in November 2023. This upgrades the project to Spring Boot 3.2 with Jakarta EE 10, Spring Security 6, and Java 17 as the minimum version. Changes: - Bump spring.boot.version from 2.6.1 to 3.2.0 - Bump java.version from 1.8 to 17 - Migrate javax.servlet, javax.annotation, javax.mail, javax.activation imports to their jakarta.* equivalents across all modules - Update MailSender mailcap handler references from com.sun.mail to org.eclipse.angus.mail (Jakarta Mail provider) - Add Jakarta Mail and Activation dependency management to root POM - Add Lombok annotation processor configuration to maven-compiler-plugin (required for annotation processing in Java 17+) - Upgrade Lombok from 1.18.20 to 1.18.36 - Upgrade maven-compiler-plugin from 3.3 to 3.11.0 - Migrate AspectJ plugin from org.codehaus.mojo to dev.aspectj 1.14.1 with aspectjtools 1.9.24 (old plugin doesn't support Java 17) - Add JVM --add-opens arguments for test compatibility - Add <release> tag to compiler plugin for Java 17 API compliance All modules compile successfully with: mvn clean compile -DskipTests -Dspotless.check.skip=true
|
Thanks for opening this pull request! Please check out our contributing guidelines. (https://github.com/apache/dolphinscheduler/blob/dev/docs/docs/en/contribute/join/pull-request.md) |
Member
|
Such a big incompatible change needs careful discussion in the community. Please don't rashly submit a similar PR directly, and I will close this PR. |
Author
My apologies! With go through the proper steps next time! |
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.
Was this PR generated or assisted by AI?
YES — the mechanical changes (javax→jakarta renames, POM version bumps, Lombok annotation processor config) were generated by, a migration agent I built. The Spring Security, AspectJ, and Jakarta Mail fixes were applied by an LLM agent and reviewed manually. All changes were verified to compile successfully.
Purpose of the pull request
Closes #18109
Spring Boot 2.6.1 reached end of life in November 2023 and no longer receives security patches. This upgrades the project to Spring Boot 3.2 with Jakarta EE 10, Spring Security 6, and Java 17 as the minimum version.
Brief change log
spring.boot.versionfrom 2.6.1 to 3.2.0java.versionfrom 1.8 to 17javax.servlet,javax.annotation,javax.mail,javax.activationimports tojakarta.*equivalents (~50 Java files)com.sun.mailtoorg.eclipse.angus.mailmaven-compiler-plugin(~130 POM files)maven-compiler-pluginfrom 3.3 to 3.11.0org.codehaus.mojotodev.aspectj1.14.1 withaspectjtools1.9.24--add-opensarguments for test compatibility<release>tag to compiler plugin for Java 17 API complianceVerify this pull request
This pull request is already covered by existing tests.
All modules compile successfully:
mvn clean compile -DskipTests -Dspotless.check.skip=true # BUILD SUCCESSVideo Walkthrough
prexplainer-tour-2026-03-31T19-52-05.mp4
Created using PRExplainer