Use node_redis v0.8.3 or remove explicit version dependency

[prashantrajan]

Any chance we could bump up the node_redis dependency to the latest version or remove the explicit version dependency altogether?

Thanks for building this lib!
I've been evaluating this library and it seems to work as expected (basic use case of failover to a slave when master dies).
I really like this lightweight approach as compared to the heavier solutions mentioned in redis/node-redis#302

I'm wondering if you have used this lib in a production environment? Any issues to be aware of?

[jamessharp]

Thanks. I'd like to keep the version explicit for now because this library relies on node_redis's internals working in a particular way.

I've been using this in production for a while now and it's been fine. Saying that I don't think it's actually had to failover yet... The main issues are as mentioned in the issue - there will be some downtime on failover and requests sent during this period will be lost (as will any pub/subs)

[prashantrajan]

Thanks for merging.
I've started using this lib in a production env - works as expected. Good stuff!

[prashantrajan]

Is pub/sub not supported at all or it breaks when the failover occurs? That wasn't clear from the issue.

[jamessharp]

It will break when the failover occurs. Otherwise it should be fine.

On 17 May 2013, at 10:58, Prashant Nadarajan notifications@github.com wrote:

Is pub/sub not supported at all or it breaks when the failover occurs? That wasn't clear from the issue.

—
Reply to this email directly or view it on GitHub.