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

How to use with sentinel? #48

Open
richmolj opened this issue Dec 4, 2015 · 2 comments
Open

How to use with sentinel? #48

richmolj opened this issue Dec 4, 2015 · 2 comments

Comments

@richmolj
Copy link

richmolj commented Dec 4, 2015

I've been attempting to pass my own redisClient, so I could use sentinel-aware ioredis. It looks like this won't work, and the injection is only for testing, since redisClient needs decorations provided by this repo (upload, activate, etc) and it doesn't seem possible to use an alternate redis lib (since it relies on a particular method signature). Am I understanding this correctly? Anything I can do here?

@lukemelia
Copy link
Contributor

@richmolj yes, the redisClient config was created in the context of testing. You could provide your own wrapper around the ioredis, though, which adapts the required method signature appropriately.

@ghedamat
Copy link
Contributor

@richmolj was luke's suggestion enough? or were you required to make other changes for this to work?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants