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

Memory Leak #14

Open
websitewill opened this issue Nov 10, 2020 · 2 comments
Open

Memory Leak #14

websitewill opened this issue Nov 10, 2020 · 2 comments
Assignees

Comments

@websitewill
Copy link

I think there is a memory leak with how this is setup.
If you run the demo application and just hold the enter key (force it to do a bunch of requests in the loop) you will see in Process Explorer (or whatever you choose to use), the memory on DemoActor and DemoService increase over time and doesn't ever seem to be released.

I suspect it has something to do with the RemotingContext being static and using those AsyncLocal in the StateDictionary.
Is there something that needs to be done at the end of processing to clear out this memory? Or maybe the leak is somewhere else?

Thanks!
Will

@websitewill
Copy link
Author

To add a little detail, the memory does seem to be released, but it takes an incredibly long time - I let the app sit idle and it took about an hour to get back to the start point of ~95K each. Maybe that's just something with how service fabric handles garbage collection? Something to be concerned with/ Anything I can tune here, if not a bug?

@Expecho Expecho self-assigned this Nov 11, 2020
@Expecho
Copy link
Owner

Expecho commented Nov 12, 2020

I have to find some time to investigate this :-)

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

No branches or pull requests

2 participants