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

config command is an object not a function #20

Open
geogeim opened this issue Apr 11, 2017 · 2 comments
Open

config command is an object not a function #20

geogeim opened this issue Apr 11, 2017 · 2 comments

Comments

@geogeim
Copy link

geogeim commented Apr 11, 2017

i'm unable to call the config command to configure the redis server because (new RedisClustr()).config is not a function

@jbt
Copy link
Member

jbt commented Apr 11, 2017

Hi @geogeim - good spot that the config function is being shadowed by the config object on the instance itself.

I guess one problem with commands like CONFIG in a cluster environment is: what should they do? Since they're obviously not commands against a particular redis key, there's no natural instance in the cluster they should be sent to. Would you expect the command to be sent to every instance in the cluster?

@geogeim
Copy link
Author

geogeim commented Apr 11, 2017

i am using this method for setting flags to enable keyspace notifications
redisPub.config('SET', "notify-keyspace-events", "Ex");
in this case the command should be sent to all of the servers, but i don't think this applies to all cases. my suggestion would be to have a mechanism to enumerate the servers in the cluster and with this the user can set the configuration per server

edit: found this https://github.com/luin/ioredis#running-commands-to-multiple-nodes

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