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

make it support rediscluster #91

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

cxyxd
Copy link

@cxyxd cxyxd commented Dec 27, 2016

In you project,you ues jedis to connect redis,if our redis is a single instance,all are ok
but if our redis is a cluster,it has more then one instance,your project will get JedisMovedDataException

so,i change jedis into jediscluster at RedisSessionManager in your project
you can test it
thanks

@cxyxd cxyxd mentioned this pull request Dec 30, 2016
@danatstudy
Copy link

@cxyxd what would your context.xml look like for adding the multiple cluster nodes? I'm testing out your changes, but can't seem to nail that down, and if I just specify one of the nodes, I end up with the JedisMovedDataException.

@cxyxd
Copy link
Author

cxyxd commented Feb 23, 2017

@nothing2see
i am so sorry,I don't understand what you mean.
if you can not build rediscluster successfully or any question about rediscluster node's management you can refer http://blog.csdn.net/xu470438000/article/details/42971091

if you can not understand my project
you can refer
http://blog.csdn.net/dlf123321/article/details/53900378(this my blog )

any question? my email: [email protected]

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.

2 participants