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

[Java Client] AMQSession_0_8#isQueueExist contains dead code intending to send passive queue declaration when queue is bound to a default exchange and assert flag is set to true

    XMLWordPrintableJSON

Details

    Description

      AMQSession_0_8#isQueueExist contains dead code creating FailoverNoopSupport for sending queue declare command which never executed. The code was added when support for address based destinations on 0.8 path was introduced.
      The callback is created when ExchangeBound for default exchange returns positive reply and assert parameter is set to true. The passive queue creation flag is set in FailoverNoopSupport. Execution of FailoverNoopSupport sending passive queue declare for non-existing queue would cause session close by the broker. It seems that passive queue declaration can only fail when queue is deleted between sending of ExchangeBound and QueueDeclare. Apart from this corner case, the sending of passive queue declare should be successful and client should continue performing its work after receiving of QueueDeclareOk

      It seems that FailoverNoopSupport should be allowed to execute

      Attachments

        Activity

          People

            kwall Keith Wall
            orudyy Alex Rudyy
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: