Qpid
  1. Qpid
  2. QPID-4519

[Java broker] Virtualhost JMX MBean StatusEnabled attribute has the wrong value, and checking it should not cause an update

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.18, 0.20
    • Fix Version/s: 0.20
    • Component/s: Java Broker
    • Labels:
      None

      Description

      The VirtualHost JMX MBean is currently returning false for isStatusEnabled; it should be true as the feature is now always enabled (the attribute was only maintained for compatibility). More of an issue however is that checking the value prompts a stats update to occur, which it should not.

        Activity

        Hide
        Robbie Gemmell added a comment -

        Fix committed in http://svn.apache.org/viewvc?rev=1424763&view=rev

        Alex, can you review this change please? I would like to request it for 0.20 if there is time, given that it is a highly isolated and rather trivial change.

        Show
        Robbie Gemmell added a comment - Fix committed in http://svn.apache.org/viewvc?rev=1424763&view=rev Alex, can you review this change please? I would like to request it for 0.20 if there is time, given that it is a highly isolated and rather trivial change.
        Hide
        Alex Rudyy added a comment -

        The change looks good for me.

        As it is trivial, it will be safe to include it into 0.20 release.

        Show
        Alex Rudyy added a comment - The change looks good for me. As it is trivial, it will be safe to include it into 0.20 release.
        Hide
        Justin Ross added a comment -

        Reviewed by Alex. Approved for 0.20.

        Show
        Justin Ross added a comment - Reviewed by Alex. Approved for 0.20.
        Hide
        Robbie Gemmell added a comment -

        Now merged to the 0.20 release branch.

        Show
        Robbie Gemmell added a comment - Now merged to the 0.20 release branch.

          People

          • Assignee:
            Alex Rudyy
            Reporter:
            Robbie Gemmell
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development