search

gosexy / redis

Redis client for Go that maps the full redis command list into equivalent Go functions.
MIT License
167 stars 44 forks source link

BRPOP doesn't work for first key #7

Closed bzou closed 11 years ago

bzou commented 11 years ago

Go 1.1 + osx 10.8.4 conn is a redis client connection:

conn.LPush("list1", 1,2,3) conn.BRPop(0, "list1")

The BRPop call will not find the already exist value and block forever, it will only work if "list1" is NOT the first key in the call. The following modification will work: conn.LPush("list1", 1,2,3) conn.BRPop(0, "", "list1")

xiam commented 11 years ago

We've just closed a related issue, #5, could you try to update and try again?

bzou commented 11 years ago

You are correct. I use GO GET which doesn't include the fix yet. Thanks!

xiam commented 11 years ago

Please try go get -u to tell go get to update the package.

Regards,

José Carlos

On Jul 2, 2013, at 9:45, bzou notifications@github.com wrote:

You are correct. I use GO GET which doesn't include the fix yet. Thanks!

— Reply to this email directly or view it on GitHubhttps://github.com/gosexy/redis/issues/7#issuecomment-20349901 .