----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3388/#review4195 -----------------------------------------------------------
Looks good! This only covers the broker side. It would be ideal to also remove the client side support as it will now be entirely useless. The original inclusion of the feature was: http://svn.apache.org/viewvc?view=revision&revision=738247 which should help identifying the code to remove. (Tracking use of qpid::client::SessionImpl::sendMsgCredit would perhaps also be a good entry point). ALso worth getting Andrew Stitcher to review as the original author. - Gordon On 2012-01-04 21:18:13, Kenneth Giusti wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/3388/ > ----------------------------------------------------------- > > (Updated 2012-01-04 21:18:13) > > > Review request for qpid and Gordon Sim. > > > Summary > ------- > > The 0.14 release deprecates the max-session-rate feature. This patch removes > the code from the C++ broker. > > > This addresses bug qpid-3630. > https://issues.apache.org/jira/browse/qpid-3630 > > > Diffs > ----- > > /trunk/qpid/cpp/src/qpid/broker/Broker.h 1227163 > /trunk/qpid/cpp/src/qpid/broker/Broker.cpp 1227163 > /trunk/qpid/cpp/src/qpid/broker/ConnectionHandler.cpp 1227163 > /trunk/qpid/cpp/src/qpid/broker/ConnectionState.h 1227163 > /trunk/qpid/cpp/src/qpid/broker/RateFlowcontrol.h 1227163 > /trunk/qpid/cpp/src/qpid/broker/SessionState.h 1227163 > /trunk/qpid/cpp/src/qpid/broker/SessionState.cpp 1227163 > /trunk/qpid/cpp/src/qpid/cluster/Connection.cpp 1227163 > /trunk/qpid/cpp/src/tests/CMakeLists.txt 1227163 > /trunk/qpid/cpp/src/tests/Makefile.am 1227163 > /trunk/qpid/cpp/src/tests/RateFlowcontrolTest.cpp 1227163 > > Diff: https://reviews.apache.org/r/3388/diff > > > Testing > ------- > > cmake, automake, and unit tests. > > > Thanks, > > Kenneth > >
