Qpid
  1. Qpid
  2. QPID-2418

Existing durable subscription with selector is not unsubscribed during change to new subscription

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: M4, 0.5, 0.6
    • Fix Version/s: 0.7
    • Component/s: Java Client
    • Labels:
      None

      Description

      AMQSession.createDurableSubscriber(topic, name, messageSelector, noLocal) does not unsubscribe existing durable subscriptions. Whilst it does check for existing durable subscriptions in use by the client with the same name, it instead simply closes the subscriptions then creates a new one. As a result of not unsubscribing, the queue backing the subscription is not deleted before being used by the updated subscription as it should be (and as happens in the 0_8 and 0_10 subclasses when using durable subscriptions without selectors).

      1. QPID-2418-trunk_robbies_additions.patch
        21 kB
        Robbie Gemmell
      2. QPID-2418-trunk_new.patch
        50 kB
        Robbie Gemmell
      3. 0001-QPID-2418-trunk.patch
        53 kB
        Andrew Kennedy
      4. 0001-QPID-2418-branch.patch
        51 kB
        Andrew Kennedy

        Activity

        Marnie McCormack made changes -
        Status Ready To Review [ 10006 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Robbie Gemmell made changes -
        Status In Progress [ 3 ] Ready To Review [ 10006 ]
        Robbie Gemmell made changes -
        Attachment QPID-2418-trunk_new3.patch [ 12453361 ]
        Robbie Gemmell made changes -
        Attachment QPID-2418-trunk_new2.patch [ 12453086 ]
        Robbie Gemmell made changes -
        Attachment QPID-2418-trunk_robbies_additions.patch [ 12453904 ]
        Robbie Gemmell made changes -
        Attachment QPID-2418-trunk_new3.patch [ 12453361 ]
        Robbie Gemmell made changes -
        Attachment QPID-2418-trunk_new2.patch [ 12453086 ]
        Robbie Gemmell made changes -
        Attachment QPID-2418-trunk_new.patch [ 12451535 ]
        Andrew Kennedy made changes -
        Attachment 0001-QPID-2418-trunk.patch [ 12440522 ]
        Andrew Kennedy made changes -
        Attachment 0001-QPID-2418-trunk.patch [ 12440521 ]
        Andrew Kennedy made changes -
        Attachment 0001-QPID-2418-branch.patch [ 12440520 ]
        Attachment 0001-QPID-2418-trunk.patch [ 12440521 ]
        Andrew Kennedy made changes -
        Attachment 0001-QPID-2418-trunk.patch [ 12440512 ]
        Andrew Kennedy made changes -
        Attachment 0001-QPID-2418-branch.patch [ 12440510 ]
        Andrew Kennedy made changes -
        Attachment 0001-QPID-2418-trunk.patch [ 12440512 ]
        Andrew Kennedy made changes -
        Attachment 0001-QPID-2418-trunk.patch [ 12440511 ]
        Andrew Kennedy made changes -
        Attachment 0001-QPID-2418-branch.patch [ 12440510 ]
        Attachment 0001-QPID-2418-trunk.patch [ 12440511 ]
        Andrew Kennedy made changes -
        Attachment 0001-QPID-2418-trunk.patch [ 12440349 ]
        Andrew Kennedy made changes -
        Attachment 0001-QPID-2418-branch.patch [ 12440348 ]
        Andrew Kennedy made changes -
        Attachment 0001-QPID-2418-branch.patch [ 12440348 ]
        Attachment 0001-QPID-2418-trunk.patch [ 12440349 ]
        Andrew Kennedy made changes -
        Attachment 0001-QPID-2418-trunk.patch [ 12440322 ]
        Andrew Kennedy made changes -
        Attachment 0001-QPID-2418-branch.patch [ 12440321 ]
        Andrew Kennedy made changes -
        Attachment 0001-QPID-2418-branch.patch [ 12440321 ]
        Attachment 0001-QPID-2418-trunk.patch [ 12440322 ]
        Andrew Kennedy made changes -
        Attachment 0001-QPID-2418-trunk.patch [ 12440222 ]
        Andrew Kennedy made changes -
        Attachment 0001-QPID-2418-branch.patch [ 12440221 ]
        Andrew Kennedy made changes -
        Attachment 0001-QPID-2418-branch.patch [ 12440221 ]
        Attachment 0001-QPID-2418-trunk.patch [ 12440222 ]
        Andrew Kennedy made changes -
        Attachment 0001-QPID-2418-trunk.patch [ 12440209 ]
        Andrew Kennedy made changes -
        Attachment 0001-QPID-2418-branch.patch [ 12440208 ]
        Andrew Kennedy made changes -
        Attachment 0001-QPID-2418-branch.patch [ 12440208 ]
        Attachment 0001-QPID-2418-trunk.patch [ 12440209 ]
        Andrew Kennedy made changes -
        Attachment 0001-QPID-2418-Unsubscribe-existing-durable-subscription.patch [ 12439780 ]
        Andrew Kennedy made changes -
        Attachment 0001-QPID-2418-Unsubscribe-existing-durable-subscription.patch [ 12439780 ]
        Andrew Kennedy made changes -
        Attachment 0001-QPID-2418-Unsubscribe-existing-durable-subscription.patch [ 12439778 ]
        Andrew Kennedy made changes -
        Attachment 0001-QPID-2418-Unsubscribe-existing-durable-subscription.patch [ 12439778 ]
        Robbie Gemmell made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Robbie Gemmell made changes -
        Assignee Robbie Gemmell [ gemmellr ]
        Robbie Gemmell made changes -
        Field Original Value New Value
        Summary AMQSession.createDurableSubscriber(topic, name, messageSelector, noLocal) does not unsubscribe existing durable subscriptions  Existing durable subscription with selector is not unsubscribed during change to new subscription
        Description AMQSession.createDurableSubscriber(topic, name, messageSelector, noLocal) does not unsubscribe existing durable subscriptions. Whilst it does check for existing durable subscriptions in use by the client with the same name, it instead simply clsoes the subscriptions then creates a new one. As a result of not unsibscribing, the queue backing the subscription is not deleted before being used by the new subscription as it should be (and as happens in the 0_8 and 0_10 subclasses when using durable subscriptions without selectors). AMQSession.createDurableSubscriber(topic, name, messageSelector, noLocal) does not unsubscribe existing durable subscriptions. Whilst it does check for existing durable subscriptions in use by the client with the same name, it instead simply closes the subscriptions then creates a new one. As a result of not unsubscribing, the queue backing the subscription is not deleted before being used by the updated subscription as it should be (and as happens in the 0_8 and 0_10 subclasses when using durable subscriptions without selectors).
        Robbie Gemmell created issue -

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development