qpid-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alan Conway" <acon...@redhat.com>
Subject Review Request: QPID-3384: Enable DTX transactions in a cluster.
Date Wed, 24 Aug 2011 21:12:24 GMT

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

Review request for qpid, Gordon Sim and Kenneth Giusti.


Summary
-------

QPID-3384: Enable DTX transactions in a cluster.

Functionality appears to work but I'm not happy that its sufficiently tested.  I'd welcome
suggestions on what else could be tested and how we can automate more realistic system tests
for use of DTX.

NOTE: there are a bunch of gratuitous whitespace changes, just click "ignore whitespace changes"
at the top of the diff.


Diffs
-----

  /trunk/qpid/cpp/include/qpid/Msg.h 1161178 
  /trunk/qpid/cpp/rubygen/amqpgen.rb 1161178 
  /trunk/qpid/cpp/src/Makefile.am 1161178 
  /trunk/qpid/cpp/src/qpid/Msg.cpp PRE-CREATION 
  /trunk/qpid/cpp/src/qpid/broker/Broker.cpp 1161178 
  /trunk/qpid/cpp/src/qpid/broker/DtxAck.h 1161178 
  /trunk/qpid/cpp/src/qpid/broker/DtxAck.cpp 1161178 
  /trunk/qpid/cpp/src/qpid/broker/DtxBuffer.h 1161178 
  /trunk/qpid/cpp/src/qpid/broker/DtxBuffer.cpp 1161178 
  /trunk/qpid/cpp/src/qpid/broker/DtxManager.h 1161178 
  /trunk/qpid/cpp/src/qpid/broker/DtxManager.cpp 1161178 
  /trunk/qpid/cpp/src/qpid/broker/DtxTimeout.h 1161178 
  /trunk/qpid/cpp/src/qpid/broker/DtxTimeout.cpp 1161178 
  /trunk/qpid/cpp/src/qpid/broker/DtxWorkRecord.h 1161178 
  /trunk/qpid/cpp/src/qpid/broker/DtxWorkRecord.cpp 1161178 
  /trunk/qpid/cpp/src/qpid/broker/SemanticState.h 1161178 
  /trunk/qpid/cpp/src/qpid/broker/SemanticState.cpp 1161178 
  /trunk/qpid/cpp/src/qpid/broker/TxBuffer.cpp 1161178 
  /trunk/qpid/cpp/src/qpid/broker/TxPublish.h 1161178 
  /trunk/qpid/cpp/src/qpid/cluster/Cluster.cpp 1161178 
  /trunk/qpid/cpp/src/qpid/cluster/Connection.h 1161178 
  /trunk/qpid/cpp/src/qpid/cluster/Connection.cpp 1161178 
  /trunk/qpid/cpp/src/qpid/cluster/UpdateClient.h 1161178 
  /trunk/qpid/cpp/src/qpid/cluster/UpdateClient.cpp 1161178 
  /trunk/qpid/cpp/src/qpid/cluster/UpdateReceiver.h 1161178 
  /trunk/qpid/cpp/src/qpid/log/Statement.cpp 1161178 
  /trunk/qpid/cpp/src/tests/brokertest.py 1161178 
  /trunk/qpid/cpp/src/tests/cluster_python_tests_failing.txt 1161178 
  /trunk/qpid/cpp/src/tests/cluster_tests.py 1161178 
  /trunk/qpid/cpp/xml/cluster.xml 1161178 

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


Testing
-------

new unit tests, make check , qpid-txtest


Thanks,

Alan


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