-
Notifications
You must be signed in to change notification settings - Fork 149
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Jakarta ee 8 integration #900
Merged
Merged
Conversation
This file contains 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
back jax-rs for jak ee8
* more errors in changes * clean gradle * add injection hk2 dep
Update dependency for cxf-2.7
Update servlet instrumentation dependency and verifier
Servlet user
Update dependency and verifier for jsp-2.4
Update dependency and verifier for ejb-3.0
Update jms dependency and verifier
Update wildfly CAT dependency
Wildfly port jakarta
Update wildfly-8 dependencies
Tomcat request listener jakarta
Update tomcat-8.5.2 dependencies
Update glassfish dependecies and verifier
Update jboss dependencies and verifier
* revert jetty version change * fix spacing
* revert jetty version change * fix spacing * bad char
Update spring-jms dependencies and verifier
Update spring-ws dependencies and verifier
Merging main to jakarta 8
Update jetty-servlet-9 dependencies and verifier
* jetty 10 insrumentationm module * fix settings.gradle * fix typo removed a rando typo
…akarta-ee-8-integration
Update was liberty dependency
Fix order jetty-10 remove unnecessary findProject command
Jetty 10 fix
twcrone
approved these changes
Jul 22, 2022
jasonjkeller
approved these changes
Jul 22, 2022
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.
Jakarta EE phase 1
Overview
The Jakarta EE Platform 8 has the same set of specifications from Java EE 8 with no changes in its features. The only change is the new process to evolve these specifications now that ownership has been transferred from Oracle to Eclipse and open sourced.
Even though Jakarta EE 8 is feature identical to Java EE 8 it does publish artifacts under new maven coordinates: jakarta.platform:jakarta.jakartaee-api:8.0.0 (previously was javax:javaee-api:8.0.1). As can be seen in the Javadocs, all packages are still using javax.* so I think our instrumentation should still apply.
Here's the Jakarta 8 integration branches. Open all Jakarta 8 related PRs against them:
Java agent: https://github.com/newrelic/newrelic-java-agent/tree/jakarta-ee-8-integration
AITs: https://github.com/newrelic/java-agent-integration-tests/tree/jakarta-ee-8-integration
Jakarta EE 8 - Key Takeaways
Functionally equivalent to Java EE 8
Largely a release to get open source processes in place
Publishes new jakarta artifacts but doesn’t change package names
Maven coordinates
jakarta.platform:jakarta.jakartaee-api:8.0.0
Minimum runtime supported: Java SE 8