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

(Updated 2011-08-16 22:04:42.180010)


Review request for qpid.


Changes
-------

Updated to incorporate review input:

1) removed message filter from MessageAllocator, created a more generic filter
2) removed unneeded stuff


Summary
-------

Some initial refactoring of Queue/Consumer interface to allow for message 
grouping support.  Very preliminary.


This addresses bug qpid-3346.
    https://issues.apache.org/jira/browse/qpid-3346


Diffs (updated)
-----

  /branches/qpid-3346/qpid/cpp/src/qpid/broker/Queue.cpp 1158073 
  /branches/qpid-3346/qpid/cpp/src/qpid/broker/MessageDeque.cpp 1158073 
  /branches/qpid-3346/qpid/cpp/src/qpid/broker/MessageMap.h 1158073 
  /branches/qpid-3346/qpid/cpp/src/qpid/broker/MessageMap.cpp 1158073 
  /branches/qpid-3346/qpid/cpp/src/qpid/broker/Messages.h 1158073 
  /branches/qpid-3346/qpid/cpp/src/qpid/broker/PriorityQueue.h 1158073 
  /branches/qpid-3346/qpid/cpp/src/qpid/broker/PriorityQueue.cpp 1158073 
  /branches/qpid-3346/qpid/cpp/src/qpid/broker/Queue.h 1158073 
  /branches/qpid-3346/qpid/cpp/src/qpid/broker/LegacyLVQ.cpp 1158073 
  /branches/qpid-3346/qpid/cpp/src/qpid/broker/MessageDeque.h 1158073 
  /branches/qpid-3346/qpid/cpp/src/qpid/broker/LegacyLVQ.h 1158073 
  /branches/qpid-3346/qpid/cpp/src/qpid/broker/DeliveryRecord.cpp 1158073 

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


Testing
-------

minimal - passes unit tests on linux.


Thanks,

Kenneth

Reply via email to