search

alphazero / jredis

Java Client and Connectors for Redis
http://code.google.com/p/jredis/
Apache License 2.0
315 stars 136 forks source link

memory leak with HeartbeatJinn #31

Open raszi opened 13 years ago

raszi commented 13 years ago

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 commented 13 years ago

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

raszi commented 13 years ago

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

pastorius commented 13 years ago

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

alphazero commented 13 years ago

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.