----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/672/#review615 -----------------------------------------------------------
Ship it! - Kenneth On 2011-04-28 20:12:48, Alan Conway wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/672/ > ----------------------------------------------------------- > > (Updated 2011-04-28 20:12:48) > > > Review request for qpid, Kenneth Giusti and Ted Ross. > > > Summary > ------- > > QPID-3235: clustered qpidd broker fails ocassionly the > cluster_tests.ShortTests.test_route_update > > Inconsistent stats changes on a Link were causing cluster > inconsistency. Fix is to disable those stats changes in a cluster. > > Updated cluster_tests.py to reliably generate the error every time > without the fix. > > Note this is also https://bugzilla.redhat.com/show_bug.cgi?id=675921 > > > Diffs > ----- > > /trunk/qpid/cpp/src/qpid/broker/Link.h 1096751 > /trunk/qpid/cpp/src/qpid/broker/Link.cpp 1096751 > /trunk/qpid/cpp/src/qpid/broker/LinkRegistry.cpp 1096751 > /trunk/qpid/cpp/src/tests/cluster_tests.py 1096751 > > Diff: https://reviews.apache.org/r/672/diff > > > Testing > ------- > > > Thanks, > > Alan > >
