+1 Well done under fire.
Weston On Apr 19, 2011, at 1:29 PM, Gordon Sim wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/623/#review496 > ----------------------------------------------------------- > > Ship it! > > > Nice fix; thanks! > > - Gordon > > > On 2011-04-19 17:23:42, Alan Conway wrote: >> >> ----------------------------------------------------------- >> This is an automatically generated e-mail. To reply, visit: >> https://reviews.apache.org/r/623/ >> ----------------------------------------------------------- >> >> (Updated 2011-04-19 17:23:42) >> >> >> Review request for qpid and Gordon Sim. >> >> >> Summary >> ------- >> >> QPID-3215: cached exchange reference can cause cluster inconsistencies if >> exchange is deleted/recreated >> >> SemanticState::route() uses a simple cache variable to avoid looking >> up the exchange for every message. However if the exchange in question >> is deleted, even if then recreated, this can cause inconsistencies in >> a cluster. >> >> Even in a stand-alone broker messages can be routed by a deleted >> exchange because of the cache. >> >> Fix is to mark the exchange deleted and check the status when using >> the cached exchange. >> >> >> Diffs >> ----- >> >> /trunk/qpid/cpp/src/qpid/broker/Exchange.h 1095059 >> /trunk/qpid/cpp/src/qpid/broker/Exchange.cpp 1095059 >> /trunk/qpid/cpp/src/qpid/broker/ExchangeRegistry.cpp 1095059 >> /trunk/qpid/cpp/src/qpid/broker/SemanticState.cpp 1095059 >> /trunk/qpid/cpp/src/tests/cluster_tests.py 1095059 >> >> Diff: https://reviews.apache.org/r/623/diff >> >> >> Testing >> ------- >> >> >> Thanks, >> >> Alan >> >> > --------------------------------------------------------------------- Apache Qpid - AMQP Messaging Implementation Project: http://qpid.apache.org Use/Interact: mailto:dev-subscr...@qpid.apache.org