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

Update redis commands to use array notation #9

Conversation

ruimarinho
Copy link
Contributor

This is a small performance improvement since the redis module offers
a straight passthrough mode for command callbacks, since it does not
rely on parsing arguments.

@ruimarinho ruimarinho force-pushed the enhancement/use-array-args-for-redis-commands branch from dbbd1cb to fc6ea0e Compare December 31, 2014 19:49
This is a small performance improvement since the `redis` module offers
a straight passthrough mode for command callbacks, since it does not
rely on parsing `arguments`.
@ruimarinho ruimarinho force-pushed the enhancement/use-array-args-for-redis-commands branch from fc6ea0e to 8e01ba9 Compare December 31, 2014 19:56
@ruimarinho ruimarinho changed the title Update watch and mget to use array arguments Update redis commands to use array notation Dec 31, 2014
@kadishmal
Copy link
Contributor

Nice!

@ruimarinho
Copy link
Contributor Author

Any chance of taking a look at this @noamshemesh?

@noamshemesh
Copy link
Collaborator

That looks awesome! I probably will merge it on this weekend.

@noamshemesh noamshemesh merged commit 8e01ba9 into tj:master Jan 10, 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

Successfully merging this pull request may close these issues.

3 participants