MBA-306: Upgrade Java from 17 to 21 #15
Open
Conversation
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
- Update java.version property from 17 to 21 in pom.xml - Fix Maven compiler plugin configuration: update source and target from 1.8 to 21 - Upgrade Maven compiler plugin from 3.8.0 to 3.11.0 for Java 21 support - Update Docker builder image to maven:3.9.6-openjdk-21 - Update Docker runtime image to openjdk:21-alpine This addresses the critical Maven compiler plugin inconsistency where the project declared Java 17 but the compiler plugin was still targeting Java 1.8. Co-Authored-By: Jia Wu <jia.wu@cognition.ai>
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
- Change maven:3.9.6-openjdk-21 to maven:3.9.11-eclipse-temurin-21-alpine - Change openjdk:21-alpine to eclipse-temurin:21-jdk-alpine The original OpenJDK Docker images have been deprecated and are no longer available on Docker Hub. Eclipse Temurin is the official replacement for OpenJDK Docker images. Verified successful build with Java 21 via Docker. Co-Authored-By: Jia Wu <jia.wu@cognition.ai>
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.
MBA-306: Upgrade Java from 17 to 21
Summary
This PR upgrades the Spring Boot banking application from Java 17 to Java 21 to address JIRA ticket MBA-306. The changes include updating Maven configuration, fixing a critical Maven compiler plugin inconsistency, and updating Docker images.
Key Changes:
java.versionproperty from 17 to 21 in pom.xmlmaven:3.9.11-eclipse-temurin-21-alpineeclipse-temurin:21-jdk-alpineImportant Note: The original specification requested
maven:3.9.6-openjdk-21andopenjdk:21-alpineDocker images, but these don't exist on Docker Hub (OpenJDK Docker images have been deprecated). I switched to Eclipse Temurin images, which are the official replacement for OpenJDK Docker images.Review & Testing Checklist for Human
Notes
Link to Devin run: https://app.devin.ai/sessions/e73091b72b154cffb388f695d91ec2e0
Requested by: Jia Wu (jia.wu@cognition.ai)