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

Allow Snappy to be loaded from a shared classloader #40304

Merged
merged 1 commit into from
Apr 30, 2024

Conversation

cescoffier
Copy link
Member

This commit introduces a solution for loading the Snappy native library across multiple test profiles. Due to the constraint that native libraries can only be loaded from a single classloader, a shared classloader is now utilized for loading Snappy in test mode.

Please note, this feature is exclusively applicable when running tests.

Fixes: #39767

This commit introduces a solution for loading the Snappy native library across multiple test profiles. Due to the constraint that native libraries can only be loaded from a single classloader, a shared classloader is now utilized for loading Snappy in test mode.

Please note, this feature is exclusively applicable when running tests.

Fixes: quarkusio#39767
Copy link

quarkus-bot bot commented Apr 26, 2024

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit 3ffe398.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

You can consult the Develocity build scans.

@cescoffier cescoffier requested a review from geoand April 30, 2024 11:13
@cescoffier cescoffier merged commit 1323185 into quarkusio:main Apr 30, 2024
23 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.11 - main milestone Apr 30, 2024
@cescoffier cescoffier deleted the snappy-shared-classloader branch May 2, 2024 07:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Kafka Snappy library fails to load when restarting Quarkus in the same JVM
3 participants