qpid-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Gordon Sim" <g...@redhat.com>
Subject Review Request 21267: QPID-5758: Move purging of expired messages from timer thread to worker thread
Date Fri, 09 May 2014 17:00:31 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/21267/
-----------------------------------------------------------

Review request for qpid, Alan Conway and Pavel Moravec.


Bugs: QPID-5758
    https://issues.apache.org/jira/browse/QPID-5758


Repository: qpid


Description
-------

Uses a PollableQueue to have queues purged by worker threads rather than doing it on the timer
thread which may affect other tasks.


Diffs
-----

  /trunk/qpid/cpp/src/qpid/broker/Broker.cpp 1593267 
  /trunk/qpid/cpp/src/qpid/broker/QueueCleaner.h 1593267 
  /trunk/qpid/cpp/src/qpid/broker/QueueCleaner.cpp 1593267 
  /trunk/qpid/cpp/src/tests/QueueTest.cpp 1593267 

Diff: https://reviews.apache.org/r/21267/diff/


Testing
-------

make test passes; some manual testing of expiration also


Thanks,

Gordon Sim


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message