Qpid
  1. Qpid
  2. QPID-3490 the default exchange should not be implemented as just a regular direct exchange
  3. QPID-3609

Refactor implementation of the default exchanges and functionality to bind queue to default exchange

    Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Major Major
    • Resolution: Duplicate
    • Affects Version/s: 0.5, 0.6, 0.7, 0.8, 0.9, 0.10, 0.11, 0.12, 0.13, 0.14, 0.15
    • Fix Version/s: None
    • Component/s: Java Broker
    • Labels:
      None

      Description

      The implementation of the default exchange should be updated such that it is not just a regular direct exchange (e.g. to prevent it having an MBean or at least a custom MBean, since no operations should actually be able to be performed on it given it is to just be a direct mapping to the available queues), and its bindings should be handled directly through the Queue factory to ensure they are applied uniformly across the broker.

      Additionally, the current implementation of the default exchange has a name <<default>> when it should really be nameless, and it is possible to delete it (and amq.* exchanges) via the JMX interface as a result which it shouldnt be.

        Issue Links

          Activity

          Alex Rudyy created issue -
          Alex Rudyy made changes -
          Field Original Value New Value
          Status Open [ 1 ] In Progress [ 3 ]
          Alex Rudyy made changes -
          Attachment 0001-QPID-3490-creating-queue-via-JMX-now-binds-the-queue.patch [ 12503353 ]
          Alex Rudyy made changes -
          Status In Progress [ 3 ] Ready To Review [ 10006 ]
          Alex Rudyy made changes -
          Assignee Alex Rudyy [ alex.rufous ] Keith Wall [ k-wall ]
          Alex Rudyy made changes -
          Summary Refactor functionality to bind queue to default exchange Refactor implementation of the default exchanges and functionality to bind queue to default exchange
          Fix Version/s 0.15 [ 12319043 ]
          Affects Version/s 0.15 [ 12319043 ]
          Description Currently there are various places that queues are created [using the queue factory], which then require that a binding be performed to the default exchange, and in some cases this is not actually done (e.g. the createQueue JMX operation). The implementation of the default exchange should be updated such that it is not just a regular direct exchange (e.g. to prevent it having an MBean or at least a custom MBean, since no operations should actually be able to be performed on it given it is to just be a direct mapping to the available queues), and its bindings should be handled directly through the Queue factory to ensure they are applied uniformly across the broker.

          Additionally, the current implementation of the default exchange has a name <<default>> when it should really be nameless, and it is possible to delete it (and amq.* exchanges) via the JMX interface as a result which it shouldnt be.
          Alex Rudyy made changes -
          Status Ready To Review [ 10006 ] Open [ 1 ]
          Assignee Keith Wall [ k-wall ]
          Alex Rudyy made changes -
          Comment [ Keith,
          Could you please review and commit patch attached? ]
          Alex Rudyy made changes -
          Comment [ Attached a patch with task implementation ]
          Alex Rudyy made changes -
          Attachment 0001-QPID-3490-creating-queue-via-JMX-now-binds-the-queue.patch [ 12503353 ]
          Robbie Gemmell made changes -
          Assignee Robbie Gemmell [ gemmellr ]
          Robbie Gemmell made changes -
          Assignee Robbie Gemmell [ gemmellr ]
          Keith Wall made changes -
          Link This issue duplicates QPID-4832 [ QPID-4832 ]
          Keith Wall made changes -
          Status Open [ 1 ] Closed [ 6 ]
          Resolution Duplicate [ 3 ]

            People

            • Assignee:
              Unassigned
              Reporter:
              Alex Rudyy
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development