boozallen / aissemble

Booz Allen's lean manufacturing approach for holistically designing, developing and fielding AI solutions across the engineering lifecycle from data processing to model building, tuning, and training to secure operational deployment
Other
33 stars 8 forks source link

Feature: Add support for JDK 17 - Core library #133

Closed ekoniec1 closed 2 weeks ago

ekoniec1 commented 3 months ago

Description

Currently, JDK 11 is required in order to use aiSSEMBLE. OpenJDK support for 11 is ending soon and 17 is well supported across the open source community. For example, JDK 17 is compatible with Spark 3.3+ and even required for Quarkus 3.7+. Upgrading to JDK 17 will allow developers to take advantage of the latest JDK language features, performance improvements, and security fixes.

For this ticket, we will focus to make sure the build succedding until the foundation-messaging module

DOD

Test Strategy/Script

References/Additional Context

N/A

peter-mcclonski commented 3 months ago

Moved hive-metastore-service docker image from Java 11 to Java 17 as part of #127

ewilkins-csi commented 1 month ago

Prioritized #263 as a smaller bite of this work

csun-cpointe commented 2 weeks ago

DoD completed with @ewilkins-csi @nartieri @carter-cundiff

csun-cpointe commented 2 weeks ago

We decide to cut the ticket and merge in the code since we are already at the stage that we can build through the foundation module. This will also enable parallel work on the upgrade. We will flow follow on ticket to cover rest of the modules.

ewilkins-csi commented 2 weeks ago

OTS looks good!

ewilkins-csi commented 2 weeks ago

Final Test passed

  1. mvn clean install -Pintegration-test on branch feature-java17 succeeds up to and including foundation-messaging-java. (Fails at module 33/145 extensions-authzforce-attribute) Screenshot 2024-09-19 at 2 53 47 PM