qpid-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alan Conway" <acon...@redhat.com>
Subject Re: Review Request: QPID-3603: Allow Queue::setPosition() to truncate the queue.
Date Fri, 25 May 2012 13:56:33 GMT

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

(Updated 2012-05-25 13:56:33.796720)


Review request for qpid, Gordon Sim and Kenneth Giusti.


Summary
-------

    In the new HA code a backup may sometimes be ahead of the new primary after a
    fail-over. In that case the backup truncates it's queues to the same position
    as the primary so it can continue replicating.


Diffs
-----

  /trunk/qpid/cpp/src/qpid/broker/MessageDeque.h 1342442 
  /trunk/qpid/cpp/src/qpid/broker/MessageDeque.cpp 1342442 
  /trunk/qpid/cpp/src/qpid/broker/MessageMap.h 1342442 
  /trunk/qpid/cpp/src/qpid/broker/MessageMap.cpp 1342442 
  /trunk/qpid/cpp/src/qpid/broker/Messages.h 1342442 
  /trunk/qpid/cpp/src/qpid/broker/PriorityQueue.h 1342442 
  /trunk/qpid/cpp/src/qpid/broker/PriorityQueue.cpp 1342442 
  /trunk/qpid/cpp/src/qpid/broker/Queue.h 1342442 
  /trunk/qpid/cpp/src/qpid/broker/Queue.cpp 1342442 
  /trunk/qpid/cpp/src/tests/Makefile.am 1342442 
  /trunk/qpid/cpp/src/tests/QueueTest.cpp 1342442 
  /trunk/qpid/cpp/src/tests/brokertest.py 1342442 
  /trunk/qpid/tools/src/py/.gitignore 1342442 

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


Testing (updated)
-------

Passing make check, added unit tests to QueueTests.cpp


Thanks,

Alan


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