Skip to content

Switch giant to using java 21#501

Open
philmcmahon wants to merge 5 commits intomainfrom
use-java-21
Open

Switch giant to using java 21#501
philmcmahon wants to merge 5 commits intomainfrom
use-java-21

Conversation

@philmcmahon
Copy link
Contributor

@philmcmahon philmcmahon commented Jan 21, 2026

What does this change?

There's no real urgency with this change as corretto 11 is supported until 2032 but I wanted to add https://github.com/guardian/play-brotli-filter to giant, and that only supports java 21, so I'm moving giant onto java 21.

Changes:

  • Replace UseConcMarkSweepGC with UseG1GC - this has been recommended since java 9 - see https://openjdk.org/jeps/291
  • RAMFraction becomes RAMPercentage - RAMFraction still works but is deprecated

Getting this to work locally: this depends how you're managing java on your machine. I use https://github.com/jenv/jenv so I had to do:

  • brew install --cask corretto@21
  • jenv add /Library/Java/JavaVirtualMachines/amazon-corretto-21.jdk/Contents/Home
  • after this cding into the giant directory should get jenv to pick the right version - check with jenv versions

Note - this can't be released until we've fixed our AMI bakes

How to test

I've tested this on playground using this AMI: https://amigo.gutools.co.uk/recipes/investigations-giant-app-arm-j21/bakes/1

@philmcmahon philmcmahon requested a review from a team as a code owner January 21, 2026 12:41
@philmcmahon philmcmahon self-assigned this Jan 21, 2026
@philmcmahon philmcmahon added the maintenance Departmental tracking: maintenance work, not a fix or a feature label Jan 21, 2026
@philmcmahon philmcmahon force-pushed the use-java-21 branch 2 times, most recently from 8a18535 to 3c894ee Compare January 21, 2026 13:17
Base automatically changed from use-latest-panda to main January 21, 2026 13:26
Copy link
Contributor

@twrichards twrichards left a comment

Choose a reason for hiding this comment

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

LGTM (we're so 'cutting edge')

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

health maintenance Departmental tracking: maintenance work, not a fix or a feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants