Currently claimed messages can be claimed (redis)
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
zaqar |
New
|
Undecided
|
Eva Balycheva |
Bug Description
The statement in this comment is not true:
https:/
Imagine an example list of messages taken from db:
message 1: claimed
message 2: claimed
message 3: unclaimed, because claim is already expired (for example, a worker hasn't succeed in processing the message)
message 4: claimed
message 5: claimed
The comment in Zaqar code states:
"Since execution of this script can not happen in parallel, once we find the first unclaimed
message, the remaining messages will always be unclaimed as well."
But when Zaqar will find the unclaimed "message 3", it will assume that the next messages (4, 5) are unclaimed too. Zaqar will claim not only "message 3", but also "message 4" and "message 5", which is wrong.
I tested it(http://
Changed in zaqar: | |
assignee: | nobody → Eva Balycheva (ubershy) |