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 Cluster support #213

Open
ababich opened this issue Sep 21, 2016 · 5 comments
Open

Redis Cluster support #213

ababich opened this issue Sep 21, 2016 · 5 comments

Comments

@ababich
Copy link

ababich commented Sep 21, 2016

Hello! Is there any Redis Cluster support planned or tried?
First of all I mean proper group operations handling via corresponding key(s) composition and preventing incorrect attempts

May be you have some posts with having this tried/tested with Ohm or some plans about this?

Thanks!

@soveran
Copy link
Owner

soveran commented Sep 28, 2016

Hello @ababich, there are no plans for Redis Cluster support but it's something we could investigate in the future. At first sight it looks like a tough problem. We could embed key tags to make sure all the keys end up in the same node, but that kind of defeats the purpose of using Redis Cluster.

@ababich
Copy link
Author

ababich commented Oct 4, 2016

To my understanding only index-related keys should be touched
Also, LUA scripts should provide support for save actions

@soveran
Copy link
Owner

soveran commented Oct 4, 2016

I have an idea that may work. I'll get back to you tomorrow after testing it a bit.

@soveran
Copy link
Owner

soveran commented Oct 11, 2016

@ababich I've tried Ohm with Redis Cluster and it works after adding key tags. There's a branch called namespace with code that you can try, but you can also check the code online and make the changes by hand as there's nothing fancy going on. The idea is to change the name of a model, so that all keys generated include a given key tag. You can read more in the documentation for Ohm::Namespace and you can check the tests. Let me know if it works for you and maybe we can ship it.

@ababich
Copy link
Author

ababich commented Oct 11, 2016

I think it is more important to tag "relation", not model
I will scan all multi-keys operations for possible issues

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

No branches or pull requests

2 participants