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

POC - Testing MS Java 11 #25584

Closed
wezell opened this issue Jul 21, 2023 · 2 comments · Fixed by #25585 or #25892
Closed

POC - Testing MS Java 11 #25584

wezell opened this issue Jul 21, 2023 · 2 comments · Fixed by #25585 or #25892

Comments

@wezell
Copy link
Contributor

wezell commented Jul 21, 2023

Parent Issue

No response

User Story

As a dotCMS user and admin, I want dotCMS to be as efficient and performant as possible. To that end, MS just released their latest version of Java (which is running LinkedIn among other places) that includes an enhancement on escape analysis - which will allow the JVM to use scalar values where objects are not needed, e.g. int rather than Integer, thus saving on object allocations, memory and generally offering better performance. This issue a POC to see whether we can benefit from using MS's new version of java.

Here a quote from their announcement :

The release of MS Build of OpenJDK 17 and 11 now include an enhancement on escape analysis. With this feature, we identified latency improvements up to 13% on controlled benchmarks...

https://devblogs.microsoft.com/java/microsoft-build-of-openjdk-july-2023-release/

Acceptance Criteria

dotCMS docker image works normally, across a variety of platforms, including Gravitons, ARM and intel based computers.

Proposed Objective

Application Performance

Proposed Priority

Priority 2 - Important

External Links... Slack Conversations, Support Tickets, Figma Designs, etc.

No response

Assumptions & Initiation Needs

No response

Quality Assurance Notes & Workarounds

No response

Sub-Tasks & Estimates

No response

wezell added a commit that referenced this issue Jul 21, 2023
nollymar pushed a commit that referenced this issue Aug 24, 2023
Co-authored-by: Jose Castro <jose.castro@dotcms.com>
@nollymar nollymar linked a pull request Aug 24, 2023 that will close this issue
@wezell wezell linked a pull request Aug 24, 2023 that will close this issue
@nollymar nollymar self-assigned this Aug 25, 2023
@nollymar nollymar removed their assignment Aug 25, 2023
@nollymar
Copy link
Contributor

Internal QA: Passed

nollymar added a commit that referenced this issue Aug 25, 2023
* build: updating Dockerfile for our base java image

Ref: #25584

* Update README.md

---------

Co-authored-by: Nollymar Longa <nollymar.longa@dotcms.com>
@bryanboza
Copy link
Member

Basically this was tested using the demo-qa server, this is using this configuration.

@bryanboza bryanboza added QA : Approved Release : 23.09.6 Release Template Builder labels Aug 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants