Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use Java 21 runtime #364

Merged
merged 1 commit into from
Nov 28, 2024
Merged

Use Java 21 runtime #364

merged 1 commit into from
Nov 28, 2024

Conversation

bestbeforetoday
Copy link
Member

Using Java 21 as the runtime allows chaincode developers to exploit newer Java features, and provides JVM performance improvements. Java 11 is no longer supported by RedHat and public support from other vendors ends in 2027.

This change removes a test for a security manager failure since the security manager is deprecated and its use is disallowed by Java 21.

Chaincode must now use Gradle wrappers at version 8.5 or later, since that version of Gradle added support for execution using Java 21.

Using Java 21 as the runtime allows chaincode developers to exploit
newer Java features, and provides JVM performance improvements. Java 11
is no longer supported by RedHat and public support from other vendors
ends in 2027.

This change removes a test for a security manager failure since the
security manager is deprecated and its use is disallowed by Java 21.

Chaincode must now use Gradle wrappers at version 8.5 or later, since
that version of Gradle added support for execution using Java 21.

Signed-off-by: Mark S. Lewis <[email protected]>
Copy link

sonarcloud bot commented Nov 28, 2024

@bestbeforetoday bestbeforetoday marked this pull request as ready for review November 28, 2024 10:17
@bestbeforetoday bestbeforetoday requested a review from a team as a code owner November 28, 2024 10:17
@denyeart denyeart merged commit cb29b36 into hyperledger:main Nov 28, 2024
9 checks passed
@bestbeforetoday bestbeforetoday deleted the java21 branch November 28, 2024 22:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants