-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Implement transparent key prefixing #95
Comments
That will be great. We can give user an option to specify a key prefix: var redis = new Redis({ keyPrefix: 'app1:' }); and then the keys of every command sent by this instance should get a prefix of
|
We already have a method |
Sounds good. What's your opinion on giving the user flexibility for commands that have multiple keys? So something like: var redis = new Redis({keyPrefix: ['app1:', 'app2:', 'app3:']});
redis.blpop('key1', 'key2', 'key3', 0); // BLPOP app1:key1 app2:key2 app3:key3 0 Or maybe even allow the user to dynamically set the prefix? var redis = new Redis({keyPrefix: 'foo:'});
redis.blpop({keyPrefix: ['app1:', 'app2:', 'app3:']}, 'key1', 'key2', 'key3', 0); I'm not sure how useful this would be. |
I prefer to keeping this option simple. My intention to add |
Ok, that's reasonable. I'll start working on it. |
Add support for transparent key prefixing. Close #95
scanstream match option does not use this prefix |
Hi, I noticed that you have this feature on the roadmap, and I'd like to help implement it. Do you have any ideas about how it will work?
The text was updated successfully, but these errors were encountered: