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 loading custom Groovy extension modules #34446

Closed
essobedo opened this issue Jun 30, 2023 · 1 comment · Fixed by #34447
Closed

Allow loading custom Groovy extension modules #34446

essobedo opened this issue Jun 30, 2023 · 1 comment · Fixed by #34447
Labels
kind/enhancement New feature or request
Milestone

Comments

@essobedo
Copy link
Contributor

Description

To fix a memory leak, Groovy is declared as a parent first artifact in Quarkus core, consequently when Groovy tries to load the existing extension modules, it loads them from the System ClassLoader instead of the Quarkus Runtime ClassLoader so it cannot see the Groovy extension modules that are not part of Groovy itself.

The goal of this improvement would be to find another way to fix the leak so that Groovy is no more a parent-first artifact.

Implementation ideas

One way to fix both problems (leak and custom extension modules) is to simply disable the usage of ClassValue in Groovy

@essobedo
Copy link
Contributor Author

A potential fix #34447

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants