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

Backport cluster-revised branch with backward compatible way #1107

Merged
merged 9 commits into from Oct 7, 2015

Conversation

HeartSaVioR
Copy link
Contributor

Closes #987

* Changes on JedisCluster
** It uses new interfaces which are different from Jedis's interfaces
** It breaks backward compatible which shouldn't performed by minor release
* restore previous interfaces
* mark @deprecated to added methods since it'll be removed at next major
Conflicts:
	src/main/java/redis/clients/jedis/JedisCluster.java
@christophstrobl
Copy link

Thank you @HeartSaVioR!
Just plugged your branch into Spring Data Redis and the branch for cluster support there. Tests passed. Well done!

marcosnils added a commit that referenced this pull request Oct 7, 2015
Backport cluster-revised branch with backward compatible way
@marcosnils marcosnils merged commit 633c5dc into redis:2.8 Oct 7, 2015
@marcosnils
Copy link
Contributor

Let's merge this 😄

@HeartSaVioR
Copy link
Contributor Author

@marcosnils 👍 Thanks!

@christophstrobl
Copy link

thanks!

@gregturn
Copy link

gregturn commented Oct 9, 2015

👍

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

Successfully merging this pull request may close these issues.

None yet

4 participants