[MAGNOLIA-8369] Build on JDK 11 (main) Created: 01/Apr/22  Updated: 24/Jun/22  Resolved: 24/Jun/22

Status: Resolved
Project: Magnolia
Component/s: None
Affects Version/s: None
Fix Version/s: 6.3.0

Type: Story Priority: Neutral
Reporter: Michael Duerig Assignee: Michael Duerig
Resolution: Fixed Votes: 0
Labels: foundation_team
Σ Remaining Estimate: Not Specified Remaining Estimate: Not Specified
Σ Time Spent: Not Specified Time Spent: Not Specified
Σ Original Estimate: Not Specified Original Estimate: Not Specified

Issue Links:
Cloners
is cloned by ADMINCTR-136 Build on JDK 11 (admincentral) Resolved
is cloned by AWSFNDN-26 Build on JDK 11 (aws-foundation) Resolved
is cloned by CFGUI-156 Build on JDK 11 (definitions-app) Resolved
is cloned by EEPUBLISH-52 Build on JDK 11 (publish-transactional) Resolved
is cloned by EXTDAM-363 Build on JDK 11 (external-dam) Resolved
is cloned by IMGREC-106 Build on JDK 11 (image-recognition) Resolved
is cloned by LOGTOOLS-40 Build on JDK 11 (log-tools) Resolved
is cloned by MGNLCT-225 Build on JDK 11 (content-types) Resolved
is cloned by MGNLDAM-1040 Build on JDK 11 (dam) Resolved
is cloned by MGNLMAIL-133 Build on JDK 11 (mail) Resolved
is cloned by MGNLREST-430 Build on JDK 11 (rest) Resolved
is cloned by MGNLUI-7078 Build on JDK 11 (ui) Resolved
is cloned by MGNLUI-7239 Build on JDK 11 (widgetset) Resolved
is cloned by PAGES-651 Build on JDK 11 (pages) Resolved
is cloned by PSWDMNGR-33 Build on JDK 11 (password-manager) Resolved
is cloned by PUBLISHING-146 Build on JDK 11 (publishing) Resolved
is cloned by TXTREC-91 Build on JDK 11 (text-classification) Resolved
is cloned by MGNLLIC-92 Build on JDK 11 (license) Closed
is cloned by MGNLSSO-140 Build on JDK 11 (sso) Closed
is cloned by MGNLTEST-278 Build with JDK 11 on Jenkins (test-fr... Closed
Issue split
split from BUILD-313 Fix unit tests failed when compiling ... Closed
Relates
relates to DOCU-2441 Document changes in locale data from ... Open
dependency
Sub-Tasks:
Key
Summary
Type
Status
Assignee
MAGNOLIA-8371 Update to Mockito 2 Sub-task Closed Michael Duerig  
Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Release notes required:
Yes
Documentation update required:
Yes
Epic Link: Build on JDK11+

 Description   

Acceptance criteria

  • Builds on JDK 11, tests pass
  • Breaking changes are covered with test cases
  • Breaking changes are documented


 Comments   
Comment by Michael Duerig [ 01/Apr/22 ]

I did some discovery to figure out what needs to be done to bump to Java 11. See this branch. Apart from some minor adjustments there's a few areas that need more work as they might introduce some breaking changes:

  • Changes in date formatting introduced with Java 9:  Using Unicode Consortium's Common Locale Data Repository (CLDR) by default now for formatting dates. See 31afa35c. I suggest to accept the new defaults. Customers who need to stay with the old behaviour can tweak their setup via the java.locale.providers property (as is the official recommendation). See DOCU-2441.
  • Powermock woes (ff4b7096). I suggest we look into getting rid of Powermock to ease the pain going forward. See DEV-1945.
  • Some tests fail to load classes / resources (af862d82). This is caused by org.reflections.util.ClasspathHelper relying on UrlClassLoader, which is not the default class loader from JDK9 on. It appears that the org.reflections library does not work well with Java 9+ and we need to phased it out. See also DEV-1218
  • Look through the Maven build log and the Magnolia instance logs to identify further issues.
Comment by Michael Duerig [ 15/Jun/22 ]

Re-opening so we can use this ticket to track bumping the Java base version to 11

Generated at Mon Feb 12 04:32:07 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.