----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9258/#review16112 -----------------------------------------------------------
This looks ok to me. A couple of thoughts: 1) Java broker... Before you commit, run a build of the Java broker. It will probably fail in the module that handles management requests. It's fairly easy to figure out how to add a placeholder for a new attribute. 2) declaredExclusive in the queue-declare event. Is this not redundant with the "exclusive" attribute since this event is only generated upon declaration of the queue? - Ted Ross On Feb. 5, 2013, 3:33 p.m., Alan Conway wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9258/ > ----------------------------------------------------------- > > (Updated Feb. 5, 2013, 3:33 p.m.) > > > Review request for qpid, Gordon Sim, Kenneth Giusti, and Ted Ross. > > > Description > ------- > > QPID-4555: HA Add declaredExclusive management property to Queues > > The "exclusive" management property can change over the lifetime of a queue. > This creates race conditions where primary and backup disagree over whether a > queue is replicated. To resolve this, added a new management property > "declaredExclusive" which is set when the queue is created and not modified. > > Do we need to co-ordinate with the Java build over this change to the schema? > > > Diffs > ----- > > /trunk/qpid/cpp/src/qpid/broker/Broker.cpp 1441163 > /trunk/qpid/cpp/src/qpid/broker/Queue.cpp 1441163 > /trunk/qpid/cpp/src/qpid/broker/QueueRegistry.cpp 1441163 > /trunk/qpid/cpp/src/qpid/broker/QueueSettings.h 1441163 > /trunk/qpid/cpp/src/qpid/broker/QueueSettings.cpp 1441163 > /trunk/qpid/cpp/src/qpid/ha/BrokerReplicator.cpp 1441163 > /trunk/qpid/cpp/src/qpid/ha/ReplicationTest.cpp 1441163 > /trunk/qpid/cpp/src/tests/ha_tests.py 1441163 > /trunk/qpid/specs/management-schema.xml 1441163 > > Diff: https://reviews.apache.org/r/9258/diff/ > > > Testing > ------- > > make check > ha tests running 2 hours now without failure. > > > Thanks, > > Alan Conway > >