SncRedisBundle: BC break: zAdd does not support adding multiple members at once
We notices this after upgrading snc/redis-bundle (2.1.4 => 3.2.1).
In 2.1.4:
$this->get('snc_redis.mydb')->zAdd('key', 1, 'item1', 2, 'item2');
Adds both members.
In 3.2.1, same code only adds the first member.
About this issue
- Original URL
- State: closed
- Created 4 years ago
- Comments: 33
Hi, @umpirsky another phpredis developer here. This change will be in phpredis 5.2.1 (tagging 5.2.1RC1 and uploading to PECL tomorrow morning).
Edit: 5.2.1 GA will follow soon after.
Cheers! Mike
@curry684 looks like we have invalid arginfo for zadd method. I’ll fix it today and it will be a part of upcoming bugfix release
Thank you @yatsukhnenko I see https://github.com/phpredis/phpredis/commit/a8e2b021f9eb51ad3ed0cc89064e2f004c56f8ba should fix it.
Closing this here now.
So yea lazy PHPRedis connection was enabled from version
2.1.5which explains why you got no issue with2.1.4@umpirsky.Like you said @curry684, we had a similar issue: see #442 and phpredis/phpredis#1374.
Ok now it gets interesting 😄