Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Raise argument errors correctly in lua redis.call calls. #1011

Open
wants to merge 1 commit into
from

Conversation

Projects
None yet
1 participant
Contributor

c2nes commented Mar 18, 2013

This fixes issue #651 "redis.call doesn't throw lua error when passed nil arguments". I had some trouble debugging a script which came down to me inadvertently passing a boolean to a redis.call. I was confused as to why I was getting a table back from an HGET call. I understand if this is intentional, but it would be nice for the documentation to be updated in that case to be more clear.

@c2nes c2nes Raise argument errors correctly in lua redis.call calls.
Fixes issue where errors were returned as strings rather than raised as
errors in redis.call if the argument types were wrong or no arguments
were given.
4ca6330
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment