> On Oct. 1, 2013, 2:29 p.m., rajith attapattu wrote: > > Ship It!
QpidQueueOptions.java is no longer in use. Provided the QMF changes are verified by Fraser I don't foresee any other issues. - rajith ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14361/#review26555 ----------------------------------------------------------- On Oct. 1, 2013, 2:04 p.m., Alan Conway wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/14361/ > ----------------------------------------------------------- > > (Updated Oct. 1, 2013, 2:04 p.m.) > > > Review request for qpid, Fraser Adams and rajith attapattu. > > > Bugs: QPID-5197 > https://issues.apache.org/jira/browse/QPID-5197 > > > Repository: qpid > > > Description > ------- > > QPID-5197: Remove obsolete --cluster-durable/persistLastNode options in java > code. > QPID-5197: Remove obsolete --cluster-durable/persistLastNode options in C++ > and python. > > Fraser, can you look at the Java part of this and see if it is correct? > > > Diffs > ----- > > /trunk/qpid/cpp/src/qpid/client/QueueOptions.h 1528082 > /trunk/qpid/cpp/src/qpid/client/QueueOptions.cpp 1528082 > /trunk/qpid/cpp/src/tests/QueueOptionsTest.cpp 1528082 > > /trunk/qpid/doc/book/src/cpp-broker/Cheat-Sheet-for-configuring-Queue-Options.xml > 1528082 > /trunk/qpid/doc/book/src/cpp-broker/Managing-CPP-Broker.xml 1528082 > > /trunk/qpid/java/client/src/main/java/org/apache/qpid/client/messaging/address/QpidQueueOptions.java > 1528082 > /trunk/qpid/tools/src/java/bin/qpid-web/web/qmf-ui/scripts/qmf-ui.js > 1528082 > /trunk/qpid/tools/src/java/bin/qpid-web/web/ui/qmf.html 1528082 > > /trunk/qpid/tools/src/java/src/main/java/org/apache/qpid/qmf2/tools/QpidConfig.java > 1528082 > > /trunk/qpid/tools/src/java/src/main/java/org/apache/qpid/qmf2/util/GetOpt.java > 1528082 > /trunk/qpid/tools/src/py/qpid-config 1528082 > > Diff: https://reviews.apache.org/r/14361/diff/ > > > Testing > ------- > > > Thanks, > > Alan Conway > >