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

W-16685129 j17 e2e steps #259

Merged
merged 4 commits into from
Sep 23, 2024
Merged

W-16685129 j17 e2e steps #259

merged 4 commits into from
Sep 23, 2024

Conversation

sathya0
Copy link
Contributor

@sathya0 sathya0 commented Sep 9, 2024

No description provided.

@sathya0 sathya0 self-assigned this Sep 9, 2024
@sathya0 sathya0 requested review from valkyrie69 and a team as code owners September 9, 2024 17:12
modules/ROOT/pages/java-support.adoc Outdated Show resolved Hide resolved
modules/ROOT/pages/java-support.adoc Outdated Show resolved Hide resolved

Follow these steps to upgrade to Java 17:

. Learn about <<application-performance, application performance>> when upgrading to Java 17.

Choose a reason for hiding this comment

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

Why is this top of the list?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That section has important details regarding Java 17 performance considerations that users should know before doing the upgrade, like prereq info they should know.

Choose a reason for hiding this comment

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

why are the performance consideration something the users need to consider before starting? There are a lot of other activities to consider before that

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Can you please list some?

Choose a reason for hiding this comment

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

Pretty much everything else. Customers can't really evaluate the performance considerations until they actually understand the steps to take and their implications, specially when the upgrade requires changes to their applications.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'll move the step

modules/ROOT/pages/java-support.adoc Outdated Show resolved Hide resolved
modules/ROOT/pages/java-support.adoc Show resolved Hide resolved

Follow these steps to upgrade to Java 17:

. Learn about <<application-performance, application performance>> when upgrading to Java 17.

Choose a reason for hiding this comment

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

Pretty much everything else. Customers can't really evaluate the performance considerations until they actually understand the steps to take and their implications, specially when the upgrade requires changes to their applications.

@sathya0 sathya0 merged commit 442e58d into latest Sep 23, 2024
5 checks passed
@sathya0 sathya0 deleted the W-16685129-e2e-j17-sg branch September 23, 2024 20:52
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