Qpid
  1. Qpid
  2. QPID-4717

QMF issue on exchange: update timestamp doesn't seem to be getting updated

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Blocker Blocker
    • Resolution: Fixed
    • Affects Version/s: 0.21
    • Fix Version/s: 0.22
    • Component/s: C++ Broker
    • Labels:
      None

      Description

      I've just noticed an issue with QMF2 statistics on trunk (0.23).

      I'm pretty sure it didn't exist in 0.20 and below, but not sure about what's gone off to the 0.22 branch.

      The issue is that exchange object update timestamps are not being updated. The objects in question definitely have statistics updates as I'm seeing changes to msgReceives and msgRoutes but each exchange now has an _update_ts that isn't getting updated when the statistics change.

      This only seems to be affecting exchange objects, queue and connection objects still seem to be OK.

      I noticed this when I pointed the GUI at qpidd 0.23 and tried to look at the graph for qmf.default.direct msgReceives, the graph wasn't updating despite the msgReceives value updating (the graph uses the actual timestamp to derive its x index).

      I also tried adding console.log to display exchange._update_ts and I'm getting the same value each object update.

      Any idea what might have changed?

      I personally think this is a 0.22 blocker. It certainly prevents graphs of exchange statistics being displayed properly.

        Activity

        Fraser Adams created issue -
        Ted Ross made changes -
        Field Original Value New Value
        Assignee Ted Ross [ tedross ]
        Hide
        Ted Ross added a comment -

        Fixed in revision 1465068.

        Show
        Ted Ross added a comment - Fixed in revision 1465068.
        Ted Ross made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 0.23 [ 12324273 ]
        Resolution Fixed [ 1 ]
        Ted Ross made changes -
        Affects Version/s 0.21 [ 12323549 ]
        Hide
        Fraser Adams added a comment -

        Thanks Ted,
        I've just done svn update and recompiled and I'm seeing exchange stats graphs again

        Thanks for the quick turnaround!!!

        Frase

        Show
        Fraser Adams added a comment - Thanks Ted, I've just done svn update and recompiled and I'm seeing exchange stats graphs again Thanks for the quick turnaround!!! Frase
        Hide
        Justin Ross added a comment -

        Tested by Fraser. Approved for 0.22.

        Show
        Justin Ross added a comment - Tested by Fraser. Approved for 0.22.
        Ted Ross made changes -
        Fix Version/s 0.22 [ 12324272 ]
        Fix Version/s 0.23 [ 12324273 ]
        Justin Ross made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        1h 2m 1 Ted Ross 05/Apr/13 18:48
        Resolved Resolved Closed Closed
        115d 1h 4m 1 Justin Ross 29/Jul/13 19:53

          People

          • Assignee:
            Ted Ross
            Reporter:
            Fraser Adams
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development