search

3rd-Eden / node-hashring

hashring is a consistent hashing algorithm for Node.js that is compatible with libketama and python's hash_ring package
MIT License
350 stars 61 forks source link

Change cache by LRU cache to fix memory leak #9

Closed geisbruch closed 11 years ago

geisbruch commented 11 years ago

We've change the cache by an instance of simple-lru cache to fix a memory leak, We've also change str = new StringDecoder('utf8').write(str); by str = new StringDecoder('utf8').write(str.toString()); to fix test error

Please tell me if it's some thing wrong

Regards.

3rd-Eden commented 11 years ago

Looks good to me, thanks a lot for the pull request. I'll publish it tonight (GTM)

geisbruch commented 11 years ago

Excellent !!!