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

Create java-compatibility.adoc [TOMEE-2540] #486

Merged
merged 8 commits into from Jun 18, 2019

Conversation

dineshkumarg
Copy link
Contributor

This is a live document until TomEE 8.0.0 is released. Till that point of time, any updates (either additions/removal) of Java versions support can be done to this document

This is a live document until TomEE 8.0.0 is released. Till that point of time, any updates (either additions/removal) of Java versions support can be done to this document
:jbake-type: page
:jbake-status: published

TomEE 8.0.x is Java EE 7, Java 8 & Java 10. Any OS that supports Java 10 works
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

change Java EE 7 to Java EE 8

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Java 10 ended support in September 2018, there is no more release for it.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated with correct version
Thanks

modified as per review comments
thank you
For specific 8.0.0 version as of now
In future we might replace minor version with x like 8.0.x
There is a version typo
Made the changes to reflect Java 11
Copy link
Member

@Daniel-Dos Daniel-Dos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

very good for me .

thanks

@dineshkumarg
Copy link
Contributor Author

@cesarhernandezgt when you have time, can you please have a review on this and let me know if any changes are required, else please approve
Thanks

Copy link
Contributor

@cesarhernandezgt cesarhernandezgt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My only suggestion would be to change 8.0.x and 8.0.0 with 8.0.0-M3 . Since we don't have a TomEE 8.0.0 release yet and this documentation will end up been published on the website.

Also a note like the one we have on the /download page looks good:

Note: Please note the 8.0.0-M3 release is a milestone release intended for evaluation purposes and should not be used in production.

@cesarhernandezgt
Copy link
Contributor

btw, @dineshkumarg it's useful to have the JIRA ticket ID as part of the GitHub issue title to keep nice integration and easiness of navigation flow when we are reviewing PR :).

@dineshkumarg dineshkumarg changed the title Create java-compatibility.adoc Create java-compatibility.adoc [TOMEE-2540] Jun 17, 2019
@dineshkumarg
Copy link
Contributor Author

btw, @dineshkumarg it's useful to have the JIRA ticket ID as part of the GitHub issue title to keep nice integration and easiness of navigation flow when we are reviewing PR :).

taken care @cesarhernandezgt

@dineshkumarg
Copy link
Contributor Author

My only suggestion would be to change 8.0.x and 8.0.0 with 8.0.0-M3 . Since we don't have a TomEE 8.0.0 release yet and this documentation will end up been published on the website.

Also a note like the one we have on the /download page looks good:

Note: Please note the 8.0.0-M3 release is a milestone release intended for evaluation purposes and should not be used in production.

@cesarhernandezgt can you post me full URL of /download page, I couldn't edit this one https://tomee.apache.org/download-ng.html

:jbake-type: page
:jbake-status: published

TomEE 8.0.0-M3 is Java EE 8 & Java 11. Any OS that supports Java 11 works
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As far as I know, TomEE 8.0.0-M3 is still not built using Java 11.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So, TomEE 8.0.0-M3 is only Java EE 8 as of now?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But it works fine @ runtime with Java 11even if it is not build with it :)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @rzo1 , Base on a couple of thread on the mailing list [1]. Current TomEE M3 and therefore master branch doesn't fully compile nor pass yet all the test when runs on Java 11. The impression TomEE M3 runs on Java 11 may be true for a couple of apps but the server is not yet fully Java 11 compatible. That's why I think advertising on the website that TomEE M3 is Java 11 compatible may be misleading for the reader.

[1] https://lists.apache.org/list.html?dev@tomee.apache.org:lte=6M:java%2011

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cesarhernandezgt please check the document where I made changes, if the statements under Note are good. Or suggest a better notes which can be published

Thanks

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done, I marked as +1, thanks much :)

@cesarhernandezgt
Copy link
Contributor

@dineshkumarg
It must be a misspelling from my side. My above comment about the /download page is not related to updating the actual /download page, it's intended for adding the quoted text as part of the text your PR is introducing.

Copy link
Contributor

@cesarhernandezgt cesarhernandezgt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1, thanks much @dineshkumarg for the contribution.

@dblevins
Copy link
Contributor

Looks great. Really fantastic to see this kind of thing get documented and nailed down. Small but important step.

Thank you @dineshkumarg !

@dblevins dblevins merged commit bae1550 into apache:master Jun 18, 2019
@dineshkumarg
Copy link
Contributor Author

Thanks @dblevins @cesarhernandezgt @rzo1 @Daniel-Dos @gilbertoca for your help and support

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
6 participants