qpid-dev mailing list archives

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

This is an automatically generated e-mail. To reply, visit:

(Updated May 12, 2014, 2:17 p.m.)

Review request for qpid, Alan Conway and Pavel Moravec.


Use TimerTask::restart() instead of TimerTask::setupNextFire() as the former determines the
time to fire based on current time whereas latter uses the last time of last firing. In other
words this changes things to trigger a purge N seconds after the last purge completes, rather
than triggering it every N seconds regardless of when the last purge completed. (If we want
th latter behaviour, the setupNextFire() should be called in the fire() method.

Bugs: QPID-5758

Repository: qpid


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

Diffs (updated)

  /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/


make test passes; some manual testing of expiration also


Gordon Sim

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