The implementation of the command lrem is wrong.

The correct order is:
$client->lrem(QUEUE_NAME, 0, $queueId);
instead in the code is used as:
$client->lrem(QUEUE_NAME, $queueId, 0);

This error appear in the queue manager classes (Reliable and not)

CommentFileSizeAuthor
#3 redis-predis-lrem-execute-3079474-2.patch2.75 KBberdir
#2 redis-predis-lrem-execute-3079474-2.patch2.28 KBberdir

Comments

michele.lucchina created an issue. See original summary.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Active » Needs review
StatusFileSize
new redis-predis-lrem-execute-3079474-2.patch2.28 KB

Yeah, apparently the order of arguments between phpredis and predis is differently, lovely.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
StatusFileSize
new redis-predis-lrem-execute-3079474-2.patch2.75 KB

I also noticed that predis pipeline uses execute(), not exec(). One more fix for that.

  • Berdir committed aa44214 on 8.x-1.x 
    Issue #3079474 by Berdir, michele.lucchina: The implementation of the...
berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.