Uploaded image for project: 'Qpid'
  1. Qpid
  2. QPID-4173

[Java Broker] disable the subscription state operational logging by default

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 0.10, 0.12, 0.14, 0.16
    • Fix Version/s: 0.18
    • Component/s: Java Broker
    • Labels:
      None

      Description

      Disable the subscription state operational logging by default.

      This logging can be incredibly verbose if clients have a low prefetch configured, or there is a large backlog of messages on the queue, because the state will constantly flip between active and suspended as the buffer becomes full,not-full,full,not-full...etc.

      We should disable this by default as it tends to drown all the other logging that is often more useful. It will still be possible to turn the logging on at runtime using the logging management mbean (if the operational logging feature is enabled).

        Activity

        Hide
        alex.rufous Alex Rudyy added a comment - - edited

        Attached a patch resolving the issue.

        Phil,
        Could you please review it?

        Show
        alex.rufous Alex Rudyy added a comment - - edited Attached a patch resolving the issue. Phil, Could you please review it?
        Hide
        philharveyonline Philip Harvey added a comment -

        Reviewed and tested by me. Robbie - please can you commit the patch.

        Show
        philharveyonline Philip Harvey added a comment - Reviewed and tested by me. Robbie - please can you commit the patch.
        Hide
        gemmellr Robbie Gemmell added a comment -

        Patch applied.

        Show
        gemmellr Robbie Gemmell added a comment - Patch applied.
        Hide
        gemmellr Robbie Gemmell added a comment -

        Change merged to the 0.18 branch.

        Show
        gemmellr Robbie Gemmell added a comment - Change merged to the 0.18 branch.

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development