leporo / tornado-redis

Asynchronous Redis client that works within Tornado IO loop.
666 stars 162 forks source link

Redis 2.6 SRANDMEMBER extension #21

Closed Sapphire64 closed 11 years ago

Sapphire64 commented 11 years ago

I have added count param for srandmember (Redis 2.6 new feature). It works nice with Redis 2.6 but I let you know that I haven't tested updated srandmember with versions below 2.6.

Sapphire64 commented 11 years ago

А я-то думал в ветке 2.4 ничего из 2.6 нет ... :) обновляйте master уже ;)

leporo commented 11 years ago

вот, как раз пишу:

Please check the redis_2_4_support branch - a change you've proposed is already there.

I'm sorry for delays in merging it to master, I promise to take care of it as soon as be able to.

leporo commented 11 years ago

Тесты на ветке 2_4 проходят, но возможно падение скорости работы и какая-то обратная несовместимость из-за того, что я подправил порядок выполнения команд (SELECT, etc).

Отзыв приветствуется, буду меньше мандражировать, вливая в master :)

Sapphire64 commented 11 years ago

Окей, я и сам хотел спросить как можно помочь :))