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

Redis 4 Support #127

Closed
mstaack opened this issue Aug 4, 2017 · 2 comments
Closed

Redis 4 Support #127

mstaack opened this issue Aug 4, 2017 · 2 comments

Comments

@mstaack
Copy link

mstaack commented Aug 4, 2017

Whats the status of using latest redis 4 with this proxy?

Cheers Max

@doyoubi
Copy link
Contributor

doyoubi commented Aug 5, 2017

The only two ways that corvus interacts with redis are CLUSTER NODES and other data manipulating commands. Redis 4 changed its CLUSTER NODES format but after merging #124 corvus should be able to be compatible with redis 4. I've done some minor tests but we have NOT fully tested redis 4 in our production environment and new commands introduced in 3.2 and 4.0 are not supported yet for the current corvus.
We will fix the CI test and merge #124 in the comming Monday.

@mstaack
Copy link
Author

mstaack commented Aug 7, 2017

@doyoubi thx! will give a try soon

@mstaack mstaack closed this as completed Aug 7, 2017
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

2 participants