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 set to return redis response messages #6
Conversation
…a callback when nowait is set
1 similar comment
2 similar comments
$self->{redis}->wait_all_responses unless $self->{nowait}; | ||
die 'set() requires key and value arguments' unless $key && $value; | ||
|
||
if ($self->{nowait} && (!$callback || ($callback && ref $callback ne 'CODE'))) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is too strict behaviour and potentialy breaks existing application.
Even if nowait
with no callbacks, set
value is stored later, so callback is not
always required.
Cache::Redis is cache module, so in some case developers can accord delay of cache reflection.
Thank you. I add one comment. |
update set to return redis response messages
updated set() to return the Redis response messages
Also added the option to use a callback when nowait is set
and validate args for set()