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.

Sign up for GitHub

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

Jump to bottom

Memory leak in SmallRyeBatchLoaderContextProvider.INSTANCES #728

Closed
ybroeker opened this issue Apr 6, 2021 · 0 comments
Closed

Memory leak in SmallRyeBatchLoaderContextProvider.INSTANCES #728

ybroeker opened this issue Apr 6, 2021 · 0 comments

Comments

@ybroeker
Copy link
Contributor

ybroeker commented Apr 6, 2021

With each request, objects are stored in the SmallRyeBatchLoaderContextProvider.INSTANCES, but never deleted.

Depending on the number of batch operations and available memory, this leads more or less quickly to extremely slow requests (and almost 100% GC) and at some point to an OutOfMemoryError.
ybroeker added a commit to ybroeker/smallrye-graphql that referenced this issue Apr 6, 2021
@ybroeker
Fix smallrye#728: Use WeakHashMap to prevent memory leaks
df22d80
ybroeker added a commit to ybroeker/smallrye-graphql that referenced this issue Apr 6, 2021
@ybroeker
Fix smallrye#728: Use WeakHashMap to prevent memory leaks
e85d74c
phillip-kruger added a commit that referenced this issue Apr 6, 2021
@phillip-kruger
Merge pull request #729 from ybroeker/fix/memoryLeak
21fbb19 
Fix #728: Use WeakHashMap to prevent memory leaks
phillip-kruger added a commit that referenced this issue Apr 6, 2021
@phillip-kruger
Merge pull request #730 from ybroeker/backport/fix/memoryLeak
0126854 
Fix #728: Use WeakHashMap to prevent memory leaks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ybroeker