Skip to content

Update to GraalVM for JDK 21 Community #84

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

Merged
merged 2 commits into from
Nov 29, 2023
Merged

Update to GraalVM for JDK 21 Community #84

merged 2 commits into from
Nov 29, 2023

Conversation

richard-axual
Copy link
Contributor

Updated pom dependencies to Java 21
Added GraalVM Polyglot, GraalVM Python dependencies to pom.xml since they've been removed from GraalVM distribution.
Added new maven sub module to define and download GraalVM Polyglot, GraalVM Python and Truffle modules to make sure Java21 can find the modules.

Updated Docker build to:

  • use GraalVM for JDK 21
  • Copy the GraalVM, Python and Python modules to a separate directory
  • Update the entrypoint to use the upgrade-module-path option when starting KSML and DataGen

Updated Github Actions to download and use GraalVM for JDK 21 Community

Tests run

  • All unit/integration tests
  • Local Docker BuildX build as described in the readme
  • Run the example docker compose and run.sh using the local build images
  • Manually tested the updated GitHub Action, download and build OK, failed on Sonarcloud scan. This is normal for a fork build.

…hon modules and download them to a separate directory

Update Docker build to load and specify module dependency on start

Updated GitHub Action build-and-test.yml for GraalVM JDK21 download and support

Tests passed:
- All unit/integration tests
- local Docker build using readme
- Running examples using local build images
@jeroenvandisseldorp jeroenvandisseldorp merged commit 33f05f3 into Axual:main Nov 29, 2023
jeroenvandisseldorp added a commit to jeroenvandisseldorp/ksml that referenced this pull request Nov 30, 2023
* commit '33f05f3db282551d9cfd0d364d2ca56f9d7278e5':
  Update to GraalVM for JDK 21 Community (Axual#84)
  Update documentation to use new runner configurations (Axual#83)
  Update and clean up dependencies (Axual#82)

# Conflicts:
#	docs/runners.md
#	ksml-data-generator/pom.xml
#	ksml-query/pom.xml
#	ksml-runner/pom.xml
#	ksml/pom.xml
#	pom.xml
jeroenvandisseldorp added a commit to jeroenvandisseldorp/ksml that referenced this pull request Nov 30, 2023
* main:
  Update to GraalVM for JDK 21 Community (Axual#84)
  Update documentation to use new runner configurations (Axual#83)
  Update and clean up dependencies (Axual#82)

# Conflicts:
#	docs/runners.md
#	ksml-data-generator/pom.xml
@richard-axual richard-axual deleted the update-to-graalvm-jdk21 branch April 24, 2024 12:27
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