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 bug #487

Closed
deep011 opened this issue Aug 3, 2016 · 1 comment
Closed

memory leak bug #487

deep011 opened this issue Aug 3, 2016 · 1 comment

Comments

@deep011
Copy link
Contributor

deep011 commented Aug 3, 2016 •
edited
Loading

Hi, @manjuraj
We are used many mset commands at a twemproxy, and the twemproxy's memory keep increasing all the time.
I reviewed the twemproxy code carefully and find there are some mbufs were not be recovered in the redis_copy_bulk() function at some point.

And i make a pull request #486 for this bug, please help me check it. Thank you!
@andyqzb andyqzb closed this as completed Aug 4, 2016
@andyqzb
Copy link
Contributor

andyqzb commented Aug 4, 2016

fixed in PR #486

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
2 participants
@andyqzb @deep011