-
Notifications
You must be signed in to change notification settings - Fork 88
Enterprise beans 4.1 (under development) #826
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
base: master
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for jakartaee-specifications ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@tadamski thank you for creating this draft release plan. Can you update the description to use the template: https://github.com/jakartaee/specifications/blob/master/plan_review_pr_template.md and update things accordingly for the checklist? You do not appear to be a committer for Enterprise Beans, so you cannot create a new release on the project page: https://projects.eclipse.org/projects/ee4j.ejb. You will need to work with a committer in order to get that updated. |
@jhanders34 I have changed the title and mentioned lack of backwards incompatible changes. Are there any more changes necessary? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
The description (first comment) still needs to have the checklist that is in the template that I referenced. Copy the raw version of the link in my previous comment and edit the first comment to have that checklist and then checkoff the things on the list that you believe are complete. One of those checklist items will have you uncomment
and it should point to Also you will want to have the line about ratifying implementation and not just Compatible implementation. |
Note to the specification committee, the contents of this PR are still under discussion in the EJB spec group. No plan has yet to be agreed. |
@jhanders34 I have updated the description and added release record link (but for now it is invalid because I don't have the rights to create release plan). |
Hi All, why not think about standardizing this implementation present in wildfly? like this: Ejb Over http/Http2
Distributed EJB Transaction over http/http2
|
@tadamski with Tracy creating the Release Record, are you ready for the release plan to move to the mentor for review and ballot? |
@jhanders34 We've discussed and removed the EJB over HTTP feature. This is now ready to go up for vote. |
@jhanders34 yes I'm ok with release plan to move to review and ballot |
Include the following in the PR: