Qpid
  1. Qpid
  2. QPID-4158

HA transition to CATCHUP status too early.

    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

      Previously we set status to CATCHUP when the BrokerReplicators bridge was initialized. This is too early, it's possible for an aborted attempt to connect to another backup to get as far as bridge init. This patch waits till we receive the first actual message from the primary before updating status to CATCHUP.

        Activity

        Hide
        Gordon Sim added a comment -

        Merged to 0.18 release branch as r1363852.

        Show
        Gordon Sim added a comment - Merged to 0.18 release branch as r1363852.
        Hide
        Justin Ross added a comment -

        Reviewed by Gordon. Approved for 0.18.

        Show
        Justin Ross added a comment - Reviewed by Gordon. Approved for 0.18.
        Hide
        Gordon Sim added a comment -

        I'm in favour of merging this to the 0.18 branch. It is a simple fix with very limited scope and impact (all with the HA feature) and it fixes a sporadic test failure.

        Show
        Gordon Sim added a comment - I'm in favour of merging this to the 0.18 branch. It is a simple fix with very limited scope and impact (all with the HA feature) and it fixes a sporadic test failure.
        Hide
        Alan Conway added a comment -

        Fixed on trunk r1363520
        Request merge to 0.18

        Show
        Alan Conway added a comment - Fixed on trunk r1363520 Request merge to 0.18

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development