Skip to content

[test] JDK 17 upgrade#278

Draft
aaneja wants to merge 6 commits into
prestodb:masterfrom
aaneja:gradleUpgrade810_jdk17
Draft

[test] JDK 17 upgrade#278
aaneja wants to merge 6 commits into
prestodb:masterfrom
aaneja:gradleUpgrade810_jdk17

Conversation

@aaneja
Copy link
Copy Markdown
Contributor

@aaneja aaneja commented Oct 15, 2024

Tested with

  • JDK 17 temurin
  • Linux
  • ./gradlew publishToMavenLocal

Since it is not supported by default on JDK 17
See https://bugs.openjdk.org/browse/JDK-8192780 for details
Used the latest versions of below libs to get test errors to resolve
- TestNG
- ByteBuddy
- Spock

Fixed the JVM args required for tests to run successfully

All tests run successfully
- Set groupId and version for jitpack
- Add jitpack.yml
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.

1 participant