Skip to content

JDK 17 Migration

Thulasi Dass Subramanian requested to merge az/td-jdk-17-migration into master


  • Migration from Java 8 to Java 17.
  • Unit test fixes with build run args and dependency upgrades.
  • maven:3.9.3-eclipse-temurin-17 image is used as recommended by David osdu/platform/ci-cd-pipelines!957 (comment 234215)
  • For Integration Test - Refactored with Apache's CloseableHttpClient which supports all HTTP methods (include PATCH).

How to test:

Via Unit and Integration tests.

Changes include:

  • Breaking change (a change that is not backward-compatible and/or changes current functionality).

Changes in:

  • Common code

Dev Checklist:

  • Added Unit Tests, wherever applicable.
  • Updated the Readme, if applicable.
  • Existing Tests pass
  • Verified functionality locally
  • Self Reviewed my code for formatting and complex business logic.

Other comments:

  • Upgraded 'lombok' & 'jacoco' plugin version
  • Changed TestRunner 'PowerMockRunner' to 'MockitoJUnitRunner'
  • Removed 'powermock-api-mockito2' & 'powermock-module-junit4' dependencies
  • Integration test: Jersey HttpClient to support PATCH method, uses reflection logic which will not work in JDK 17. Hence replaced with Apache Apache's CloseableHttpClient.
Edited by Thulasi Dass Subramanian

Merge request reports