search

stefanwille / crystal-redis

Full featured Redis client for Crystal
MIT License
381 stars 61 forks source link

blpop not compiling #98

Closed repomaa closed 3 years ago

repomaa commented 4 years ago

There's an issue with the blpop command. When trying to use it i get the following error:

In lib/redis/src/redis/commands.cr:917:47

 917 | result[0] = without_namespace("#{result.first}")
                                               ^----
Error: undefined method 'first' for Redis::Future
repomaa commented 4 years ago

Specifically: using blpop inside a multi

stefanwille commented 4 years ago

Yes, this is a bug in a MULTI context. brpop has the same bug.

This bug is difficult to fix. I am not sure yet how I will do it.

kostya commented 3 years ago

it should compile in 2.7.0, but seems in multi it will be runtime error, can you provide example and reopen if still fails.