----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4243/ -----------------------------------------------------------
(Updated 2012-03-08 14:15:23.199271) Review request for qpid, Andrew Stitcher, Alan Conway, Gordon Sim, Ted Ross, and Steve Huston. Changes ------- Added JIRA reference Summary (updated) ------- The async store interface is intended to replace the current syncronous store interface in broker/MessageStore.h. It presents a bidirectional async interface, allowing async store operations which contain optional callbacks to be queued up for processing by the store, and for the results of these operations (if the callback is present in the operation) to be queued up for action by the broker itself. After a good deal of back-and-forth on the list, this is the latest iteration on the async store interface. Thanks to Andrew and Gordon who have provided much valuable feedback. If this interface is accepted, it will need to be implemented in two phases: 1. Broker wiring - the broker will need to be hooked into this interface using async semantics. This could be a disrptive phase, as it would likely break or disconnect the current sync interface. An async-sync adaptor would also be developed to allow current syncronous stores to continue operating under the new interface. 2. Async store development - the current async store is not currently part of the Qpid codebase as it was developed under an incompatible license (LGPL2). A new async store would be developed under Apache, with any bits moved from the old code base dual-licenced to Apache. (This will require some legal checkups.) Comments welcome. Coordinating JIRA: https://issues.apache.org/jira/browse/QPID-3858 Diffs ----- branches/asyncstore/cpp/src/CMakeLists.txt 1291264 branches/asyncstore/cpp/src/Makefile.am 1291264 branches/asyncstore/cpp/src/qpid/broker/AsyncStore.h PRE-CREATION branches/asyncstore/cpp/src/qpid/broker/AsyncStore.cpp PRE-CREATION Diff: https://reviews.apache.org/r/4243/diff Testing ------- Compiles as written, but is not implemented. Thanks, Kim
