diff options
author | Ted Ross <tross@apache.org> | 2010-09-22 12:50:58 +0000 |
---|---|---|
committer | Ted Ross <tross@apache.org> | 2010-09-22 12:50:58 +0000 |
commit | 862c6a9d25312230c6ac45fe4badfab6dd7173bb (patch) | |
tree | 05039e153142158a1dde3858048869d0298e93e4 | |
parent | a349ccd5a819226cf70618fb170785e001d42cff (diff) | |
download | qpid-python-862c6a9d25312230c6ac45fe4badfab6dd7173bb.tar.gz |
Fixed qpid-stat to skip rows when the underlying data disappears during execution.
Removed keyword arg from "sort" to support Python 2.3.
git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk@999918 13f79535-47bb-0310-9956-ffa450edef68
-rw-r--r-- | qpid/python/qpid/disp.py | 4 | ||||
-rwxr-xr-x | qpid/tools/src/py/qpid-stat | 31 |
2 files changed, 20 insertions, 15 deletions
diff --git a/qpid/python/qpid/disp.py b/qpid/python/qpid/disp.py index 1b315c9d98..c29ea009de 100644 --- a/qpid/python/qpid/disp.py +++ b/qpid/python/qpid/disp.py @@ -221,7 +221,9 @@ class Sorter: list = [] for row in rows: list.append(Sortable(row, col)) - list.sort(reverse=not inc) + list.sort() + if not inc: + list.reverse() count = 0 self.sorted = [] for row in list: diff --git a/qpid/tools/src/py/qpid-stat b/qpid/tools/src/py/qpid-stat index ae5683ec54..738f0971aa 100755 --- a/qpid/tools/src/py/qpid-stat +++ b/qpid/tools/src/py/qpid-stat @@ -412,20 +412,23 @@ class BrokerManager(Console): for oid in broker.subscriptions: s = broker.subscriptions[oid] row = [] - if self.cluster: - row.append(broker.getName()) - row.append(s.name) - row.append(self.qmf.getObjects(_objectId=s.queueRef)[0].name) - connectionRef = self.qmf.getObjects(_objectId=s.sessionRef)[0].connectionRef - row.append(self.qmf.getObjects(_objectId=connectionRef)[0].address) - row.append(self.qmf.getObjects(_objectId=connectionRef)[0].remoteProcessName) - row.append(self.qmf.getObjects(_objectId=connectionRef)[0].remotePid) - row.append(s.browsing) - row.append(s.acknowledged) - row.append(s.exclusive) - row.append(s.creditMode) - row.append(s.delivered) - rows.append(row) + try: + if self.cluster: + row.append(broker.getName()) + row.append(s.name) + row.append(self.qmf.getObjects(_objectId=s.queueRef)[0].name) + connectionRef = self.qmf.getObjects(_objectId=s.sessionRef)[0].connectionRef + row.append(self.qmf.getObjects(_objectId=connectionRef)[0].address) + row.append(self.qmf.getObjects(_objectId=connectionRef)[0].remoteProcessName) + row.append(self.qmf.getObjects(_objectId=connectionRef)[0].remotePid) + row.append(s.browsing) + row.append(s.acknowledged) + row.append(s.exclusive) + row.append(s.creditMode) + row.append(s.delivered) + rows.append(row) + except: + pass title = "Subscriptions" if self.cluster: title += " for cluster '%s'" % self.cluster.clusterName |