----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18622/#review35782 -----------------------------------------------------------
Ship it! This is a safe update to the schema. Looks good. - Ted Ross On Feb. 28, 2014, 8:34 a.m., Pavel Moravec wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/18622/ > ----------------------------------------------------------- > > (Updated Feb. 28, 2014, 8:34 a.m.) > > > Review request for qpid, Kenneth Giusti and Ted Ross. > > > Bugs: QPID-5587 > https://issues.apache.org/jira/browse/QPID-5587 > > > Repository: qpid > > > Description > ------- > > Trivial code change, raising this review mainly due to the change in > management schema, if it can't break anything. > > > Diffs > ----- > > /trunk/qpid/cpp/src/qpid/broker/Queue.h 1559494 > /trunk/qpid/cpp/src/qpid/broker/Queue.cpp 1559494 > /trunk/qpid/cpp/src/qpid/broker/management-schema.xml 1559494 > > Diff: https://reviews.apache.org/r/18622/diff/ > > > Testing > ------- > > > Thanks, > > Pavel Moravec > >
