Qpid
  1. Qpid
  2. QPID-3403

Empty bindings are not internally pruned from the direct exchange

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 0.7
    • Fix Version/s: 0.14
    • Component/s: C++ Broker
    • Labels:
      None

      Description

      Each queue that is created is automatically bound to the default (direct)
      exchange with a routing key that matches the queue name. When the queue is
      deleted, the queue is correctly removed from the binding map, but the mapping
      from routing key to queues is not removed even when there are no queues bound
      to the routing key.

      In a case where thousands of uniquely named queues are created and deleted as
      transient storage, the memory usage of the broker will continue to grow as the
      default exchange's routing key map continues to grow.

      1. QPID-3403.patch
        1.0 kB
        Gordon Sim
      2. QPID-3369.patch
        0.5 kB
        Jason Dillaman

        Issue Links

          Activity

          Show
          Justin Ross added a comment - http://svn.apache.org/viewvc?view=revision&revision=1156560
          Hide
          Gordon Sim added a comment -

          That's a good suggestion; I've added that test in.

          Show
          Gordon Sim added a comment - That's a good suggestion; I've added that test in.
          Hide
          Jason Dillaman added a comment -

          Sorry ... definitely the wrong patch. My suggested patch was basically the same as your patch. The only difference is that I had CopyOnWriteArray::empty() return true if the shared pointer wasn't initialized.

          Show
          Jason Dillaman added a comment - Sorry ... definitely the wrong patch. My suggested patch was basically the same as your patch. The only difference is that I had CopyOnWriteArray::empty() return true if the shared pointer wasn't initialized.
          Hide
          Gordon Sim added a comment -

          I think you have attached the wrong patch here?

          Suggested patch attached. Good catch btw!

          Show
          Gordon Sim added a comment - I think you have attached the wrong patch here? Suggested patch attached. Good catch btw!
          Hide
          Jason Dillaman added a comment -

          Potential patch for issue

          Show
          Jason Dillaman added a comment - Potential patch for issue

            People

            • Assignee:
              Gordon Sim
              Reporter:
              Jason Dillaman
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development