openstack/zaqar
Code Issues Proposed changes

Clarify code comment in Redis backend

Browse Source 
I noticed this says to skip *unclaimed* messages when in fact it skips
claimed ones.

Change-Id: I2173f5c8bc8cadb4dcb58e612c774acbfaa86d5b
This commit is contained in:
Ryan S. Brown 2015-06-02 16:18:36 -04:00
parent 3ff7718adb
commit 3155f443f7
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
zaqar/storage/redis/messages.py
View File

@ -236,7 +236,7 @@ class MessageController(storage.Message, scripting.Mixin):
client = self._client
if not marker and not include_claimed:
# NOTE(kgriffs): Skip unclaimed messages at the head
# NOTE(kgriffs): Skip claimed messages at the head
# of the queue; otherwise we would just filter them all
# out and likely end up with an empty list to return.
marker = self._find_first_unclaimed(queue, project, limit)