Skip to content

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