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

send_command is not a function #5

Closed
siddo420 opened this issue Jul 6, 2015 · 2 comments
Closed

send_command is not a function #5

siddo420 opened this issue Jul 6, 2015 · 2 comments

Comments

@siddo420
Copy link

siddo420 commented Jul 6, 2015

I am not sure if its an actual issue (could be a documentation bug at least).

I am currently testing metroplex and found out that its somehow dependent on the particular Redis library you're using with it (ie. https://github.com/mranney/node_redis). The reason is that it is using a function that is not supported by one of the popular Redis client libraries ioredis.

For details, see the issue I opened with ioredis --> redis/ioredis#90

@lpinca
Copy link
Member

lpinca commented Jul 6, 2015

Yes, leverage which is a dependecy of metroplex uses the send_command method of the redis client.

Maybe it makes sense to remove the leverage dependency and use the ioredis lua-scripting feature.

@lpinca
Copy link
Member

lpinca commented Jul 7, 2015

Closing this as it has been addressed in redis/ioredis@2edc5d1 and #7.

@lpinca lpinca closed this as completed Jul 7, 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

No branches or pull requests

2 participants