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

Attempt to share/deduplicate jar contents #885

Merged

Conversation

mickaelistria
Copy link

Introduce a subclass of JavacFileManager that is capable of caching and sharing jar contents.

Note that JDK content is still duplicated as it's using another filemanager that seems harder to configure.
Also note that the cached entries never get disposed at the moment.

@mickaelistria mickaelistria force-pushed the cache-jars branch 7 times, most recently from b3e1f9b to f6f2f40 Compare October 14, 2024 15:40
Introduce a subclass of JavacFileManager that is capable of caching and
sharing jar contents.
Also ensure some references are removed from context upon usage to
encourage garbage collection.

Note that JDK content is still duplicated as it's using another
filemanager that seems harder to configure, using direct call to
`PlatformUtils.lookupPlatformDescription().getFileManager()`.
Also note that the cached entries only get disposed when all
filemanagers are closed/GCed.
@mickaelistria mickaelistria merged commit d9c0beb into eclipse-jdtls:dom-with-javac Oct 14, 2024
2 of 3 checks passed
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