List & Pop of Sqlalchemy did not filter out the expired messages

Bug #1349213 reported by peng fei wang
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
zaqar
Fix Released
Undecided
peng fei wang

Bug Description

storage.sqlalchemy.messages.py : MessageController.list, pop did not filter out the expired ones.

peng fei wang (wpf)
Changed in marconi:
assignee: nobody → peng fei wang (wpf)
Revision history for this message
Feilong Wang (flwang) wrote :

Peng Fei, would you mind explaininig this bug with more detailes? Thanks. Since the expired messages will be automatically removed. Not sure if you're talking about this.

Revision history for this message
peng fei wang (wpf) wrote :

in Sqlalchemy, the expired messages are not be removed automatically, see https://bugs.launchpad.net/marconi/+bug/1331228

and they are be removed automatically, we still need to filter them out explicitly since there will a time-window

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to marconi (master)

Fix proposed to branch: master
Review: https://review.openstack.org/109897

Changed in marconi:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to marconi (master)

Reviewed: https://review.openstack.org/109897
Committed: https://git.openstack.org/cgit/openstack/marconi/commit/?id=fd2616d9dbfb486bc6d085e1ef028e940c06f459
Submitter: Jenkins
Branch: master

commit fd2616d9dbfb486bc6d085e1ef028e940c06f459
Author: pengfei wang <email address hidden>
Date: Mon Jul 28 11:13:02 2014 +0800

    Filter out expired messages

    Fix the issue that Sqlalchemy did not filter out the expired
    messages when listing & popping.

    Closes-Bug: #1349213
    Change-Id: I4d013a4bfd9e75a207fa5d2054f12e78d972eca6

Changed in marconi:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in zaqar:
milestone: none → juno-3
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in zaqar:
milestone: juno-3 → 2014.2
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.