SQLAlchemy driver does not clean up expired messages

Bug #1331228 reported by Kurt Griffiths
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
zaqar
Fix Released
Low
peng fei wang

Bug Description

The code for this needs to go into MessageController.post

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

I saw that code was already there but commented

            # cleanup all expired messages in this queue
            #self.driver.run('''
            # delete from Messages
            # where ttl <= julianday() * 86400.0 - created
            # and qid = ?''', qid)

is that Ok?

Kurt Griffiths (kgriffs)
Changed in marconi:
status: New → Triaged
importance: Undecided → Low
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/110543

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

Reviewed: https://review.openstack.org/110543
Committed: https://git.openstack.org/cgit/openstack/zaqar/commit/?id=d8904dd4959c3b5a5ba4ea84e7232bf1d97106d8
Submitter: Jenkins
Branch: master

commit d8904dd4959c3b5a5ba4ea84e7232bf1d97106d8
Author: pengfei wang <email address hidden>
Date: Wed Jul 30 13:22:00 2014 +0800

    Clean up expired messages for SQLAlchemy driver

    Change-Id: I1d14258528bdea89399d491d45e5e1dfd1b06b50
    Closes-Bug: #1331228

Changed in zaqar:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in zaqar:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in zaqar:
milestone: juno-3 → 2014.2
no longer affects: zaqar/juno
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.