Skip to content
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

Draft of the Jakarta Enterprise Beans 4.0 plan #64

Merged
merged 2 commits into from
Feb 11, 2020

Conversation

dblevins
Copy link
Contributor

@dblevins dblevins commented Feb 5, 2020

Significant detail on release plan for Jakarta Enterprise Beans 4.0. To be referenced in the Plan Review and specification page.

Signed-off-by: David Blevins <david.blevins@gmail.com>
Signed-off-by: David Blevins <david.blevins@gmail.com>
@tkburroughs
Copy link
Member

In addition to the removal of methods relying on java.security.Identity, should we also add:

  • Removal of methods relying on JAX-RPC : jakarta.ejb.SessionContext.getMessageContext()
  • Removal of deprecated method jakarta.ejb.EJBContext.getEnvironment()

Per the Jakarta EE 9 Release Plan ( https://eclipse-ee4j.github.io/jakartaee-platform/jakartaee9/JakartaEE9ReleasePlan), JAX-RPC will be pruned from Jakarta EE 9, which would mean the return type of getMessageContext() won't exist anymore.

After removing the java.security.Identity methods.... getEnvironment() will be the only remaining deprecated method throughout the EJB APIs. I think it has been deprecated since EJB 1.1. Unlike the other items being removed or made optional, I don't think this method is overly burdensome for a provider to implement... just mentioning it because it has been deprecated for a long time, and this might be the best (only) time to remove it.

@dblevins
Copy link
Contributor Author

Going to merge and create a separate PR for the JAX-RPC and Environment updates.

@dblevins dblevins merged commit 704e41f into jakartaee:master Feb 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants