Qpid
  1. Qpid
  2. QPID-4128

HA should not use amq.failover for replication links

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.17
    • Fix Version/s: 0.18
    • Component/s: C++ Clustering
    • Labels:
      None

      Description

      HA replication links should not use the amq.failover exchange to get updates reconnect targets. amq.failover provides the client failover list, HA manages a separate failover list for brokers. Replication links should be using the broker list, and not allow it to be overwritten by amq.failover updates.

      Review requested for 0.18, 2 line fix.

        Activity

        Alan Conway created issue -
        Alan Conway made changes -
        Field Original Value New Value
        Description HA replication links should not use the amq.failover exchange to get updates reconnect targets. amq.failover provides the client failover list, HA manages a separate failover list for brokers. Replication links should be using the broker list, and not allow it to be overwritten by amq.failover updates.

        Review requested for 0.18, 2 link fix.
        HA replication links should not use the amq.failover exchange to get updates reconnect targets. amq.failover provides the client failover list, HA manages a separate failover list for brokers. Replication links should be using the broker list, and not allow it to be overwritten by amq.failover updates.

        Review requested for 0.18, 2 line fix.
        Alan Conway made changes -
        Comment [ Fixed on trunk
        ------------------------------------------------------------------------
        r1360218 | aconway | 2012-07-11 11:02:36 -0400 (Wed, 11 Jul 2012) | 2 lines

        QPID-4128: Remove use of intrusive_ptr::reset, not available in older boost versions.

        ------------------------------------------------------------------------

        Requesting review for 0.18 branch. ]
        Alan Conway made changes -
        Comment [ Mixed 2 issues by mistake on same JIRA, but both are trivial 1 line fixes:
        - HA replication should not use amq.failover exchange: reviewed by gsim https://reviews.apache.org/r/5069/
        - Remove use of intrusive_ptr::reset from ha/Primary.cpp - raised by Steve Huston

        ]
        Alan Conway made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Justin Ross made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Alan Conway
            Reporter:
            Alan Conway
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development