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 with HeartbeatJinn #31

Open
raszi opened this issue Sep 14, 2010 · 4 comments
Open

memory leak with HeartbeatJinn #31

raszi opened this issue Sep 14, 2010 · 4 comments

Comments

@raszi
Copy link

raszi commented Sep 14, 2010

if the JRedisClient fails to connect to the server every connection attempt leaves a HeartbeatJinn in memory.

Because a ClientRuntimeException is thrown and the Thread will remain in the memory because nobody will stop it.

@alphazero
Copy link
Owner

I haven't pushed the patches for that yet -- its done and up fairly shortly. Thank you for reporting!

@raszi
Copy link
Author

raszi commented Oct 18, 2010

could you give me more information when will be the fix pushed into the master? thanks!

@pastorius
Copy link

@alphazero, Any word on the status of this patch? Is it really still pending?

@alphazero
Copy link
Owner

Heartbeat has been disabled for quite a while now.
However, the plumbing for event propagation is in a branch somewhere on one of my laptops :) I may get added but its a pain given then pulls that I took recently.

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

3 participants