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

Possible memory leak when sending large messages via web sockets #1202

Closed
devxplorer opened this issue Dec 20, 2018 · 7 comments
Closed

Possible memory leak when sending large messages via web sockets #1202

devxplorer opened this issue Dec 20, 2018 · 7 comments

Comments

@devxplorer
Copy link

This issue is the continuation of #1181.

Current leak can be easily reproduced by following steps in readme of test project. Please note that the branch is different - big_payload.

The result of the steps will be a graph on which you can clearly see that the memory starts to grow in spurts and is not released completely (example graph).

The test environment is the same as in the previous issue.

@andrewgodwin
Copy link
Member

OK - note that it may be a few months before I can get to this, so if anyone else wants to take a look they're welcome.

@devxplorer Have you confirmed this problem is in Channels rather than the Redis layer?

@devxplorer
Copy link
Author

@andrewgodwin

@devxplorer Have you confirmed this problem is in Channels rather than the Redis layer?

I can not say for sure, could you please tell how can I find it out?

@devxplorer
Copy link
Author

Strange thing, I tried to run a test script on another computer, there was no memory leak on the resulting graph. I will try to find out under what conditions it occurs.

@andrewgodwin
Copy link
Member

You can turn off the Redis channel layer (set CHANNEL_LAYERS to empty), make sure none of your code uses it, and test again.

@agronick
Copy link

agronick commented Jan 1, 2019

@devxplorer you sure you updated all your channels packages?

@devxplorer
Copy link
Author

devxplorer commented Jan 6, 2019

@devxplorer you sure you updated all your channels packages?

Maybe not. But now I can not repeat the result which is on the graph neither with the old versions of libraries, nor with the new ones.

@carltongibson
Copy link
Member

I'm going to close this. If something is still reproducible here please open a fresh issue.

@carltongibson carltongibson closed this as not planned Won't fix, can't repro, duplicate, stale Apr 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants