Skip to content

Conversation

suyash-jai
Copy link

@suyash-jai suyash-jai commented Sep 29, 2025

Description

As part of the migration from JDK 8 to JDK 17, the environment variables have been updated to ensure the application runs using Java 17. This change is required because the latest jars are compiled with JDK 17 and will not be compatible with JDK 8. The update sets JAVA_HOME, adjusts the PATH, and defines JAVA_BIN accordingly.

Changes

Updated JAVA_HOME to point to JDK 17 (/usr/lib/jvm/java-17-openjdk-amd64).
Updated PATH to ensure Java 17 executables are used by default.
Defined JAVA_BIN explicitly as $JAVA_HOME/bin/java.
Ensures compatibility with jars compiled using JDK 17.

While upgrading from JDK 8 to JDK 17, reflective access to certain internal Java APIs (like java.lang and java.lang.reflect) is restricted due to JDK 9+ module system changes. To maintain backward compatibility with existing code and dependencies that rely on reflection, the following JVM parameters have been added:

--add-opens=java.base/java.lang=ALL-UNNAMED

--add-opens=java.base/java.lang.reflect=ALL-UNNAMED

Testing Done

Created an engine with these changes.
Verified that the latest version is successfully verified and upgraded after applying the changes.

@suyash-jai suyash-jai force-pushed the dlpx/pr/suyash-jai/d292acd9-1256-41ed-99eb-8b2a7d51e01f branch 2 times, most recently from 50ec6f0 to 344536d Compare September 30, 2025 06:34
@suyash-jai suyash-jai self-assigned this Sep 30, 2025
@suyash-jai suyash-jai marked this pull request as ready for review September 30, 2025 12:43
@suyash-jai suyash-jai force-pushed the dlpx/pr/suyash-jai/d292acd9-1256-41ed-99eb-8b2a7d51e01f branch from 344536d to 9054aff Compare October 10, 2025 09:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant