----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25938/#review54267 -----------------------------------------------------------
Does this work if 1.0 is not supported? My initial instinct I think would be to populate a qmf::org::apache::qpid::broker::Subscription object for the outgoing link, rather than doing it in the tool. - Gordon Sim On Sept. 23, 2014, 12:48 p.m., Pavel Moravec wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/25938/ > ----------------------------------------------------------- > > (Updated Sept. 23, 2014, 12:48 p.m.) > > > Review request for qpid, Ernie Allen and Gordon Sim. > > > Bugs: QPID-6113 > https://issues.apache.org/jira/browse/QPID-6113 > > > Repository: qpid > > > Description > ------- > > A straightforward extension of "qpid-stat -u" to include AMQP 1.0 Outgoing > objects to have all consumers of queues in one output. > > Some minor questions/issues: > > - columns "browsing", "acknowledged" and "exclusive" would remain False > everytime for Outgoging objects, as these attributes are irrelevant or QMF > object Outgoing does not track it > - credit mode is not tracked in Outgoing object as well, as that string is > filled everytime, I suggest to have there AMQP1.0 or anything similar - this > allows to distinguish Subscriptions from outgoings as well > > I could extend Outgoing QMF object to cover e.g. browsing or creditMode > Subscription property also in Outgoing, I am just not sure how much > beneficial it would be. > > > Diffs > ----- > > trunk/qpid/tools/src/py/qpid-stat 1626808 > trunk/qpid/tools/src/py/qpidtoollibs/broker.py 1626808 > > Diff: https://reviews.apache.org/r/25938/diff/ > > > Testing > ------- > > Example output: > > > ./tools/src/py/qpid-stat -u > Subscriptions and Outgoings > subscr queue > conn procName procId browse > acked excl creditMode delivered sessUnacked > > ====================================================================================================================================================================================================== > 0 > 620f2fc9-9b4d-4593-858e-5b13347c2657:0.0 qpid.[::1]:5672-[::1]:52684 > qpid-stat 768 CREDIT 0 0 > q q > qpid.127.0.0.1:5672-127.0.0.1:46453 qpid-receive 517 > Y WINDOW 0 0 > q_3114e33b-dd76-491b-8f9c-0b4d50bcd24d q > qpid.127.0.0.1:5672-127.0.0.1:46623 qpid-receive 710 > AMQP1.0 0 0 > > > Thanks, > > Pavel Moravec > >