Skip to content

JAVA 27 FAT - #77

Merged
Arunkumar-Kallyodan merged 4 commits into
OpenLiberty:mainfrom
Arunkumar-Kallyodan:FAT_Java_27
Aug 5, 2026
Merged

JAVA 27 FAT#77
Arunkumar-Kallyodan merged 4 commits into
OpenLiberty:mainfrom
Arunkumar-Kallyodan:FAT_Java_27

Conversation

@Arunkumar-Kallyodan

Copy link
Copy Markdown
Contributor

Add Java 27 FAT: JEP 527, JEP 534, JEP 536

 Co-authored-by-AI: IBM Bob
Comment thread io.openliberty.java.internal_fat_27/src/main/java/module-info.java Outdated
Comment thread io.openliberty.java.internal_fat_27/build.gradle Outdated
Comment thread io.openliberty.java.internal_fat_27/build.gradle
Comment thread io.openliberty.java.internal_fat_27/run/jvm.options
// https://openjdk.org/jeps/534
//
// Makes compact object headers (64 bits / 8 bytes, down from 96 bits / 12 bytes)
// the default on 64-bit HotSpot. Controlled by -XX:+/-UseCompactObjectHeaders.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

suggestion: why not add -XX:+UserCompactObjectHeaders to the jvm.options, then actually verify that an empty object is 8 bytes instead of 12 bytes? This seems like a more functional test then just testing to see if the UserCompactObjectHeaders option was set or not with no failure case.
However, if that is out of scope of this issue I'll approve it the way it is.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Agreed, setting it in jvm.options could makes the FAT self-contained and immune to any container-level override.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Adding a soft notice into a hard failure as well

IBM Vault Radar pattern-matched password=sup3rS3cr3t! — a word that looks like a real password assignment. Changed the value as

@KyleAure KyleAure left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good, though there is a merge conflict you'll need to resolve before merging.

@Arunkumar-Kallyodan
Arunkumar-Kallyodan marked this pull request as ready for review August 5, 2026 07:14
@Arunkumar-Kallyodan
Arunkumar-Kallyodan merged commit 8a4894d into OpenLiberty:main Aug 5, 2026
11 checks passed
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.

2 participants