ActiveMQ
  1. ActiveMQ
  2. AMQ-3513

initialReconnectDelay is ignored for Failover transport

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 5.5.0
    • Fix Version/s: 5.6.0
    • Component/s: Transport
    • Labels:
      None

      Description

      initalReconnectDelay is ignored during failover. If multiple brokers are running, the failover transport attempts to failover immediately and does not honor the initialReconnect delay. Patch is attached.

      1. AMQ-3513-take3.patch
        12 kB
        Jeff Genender
      2. AMQ-3513-take2.patch
        12 kB
        Jeff Genender
      3. AMQ-3513.patch
        5 kB
        Jeff Genender

        Activity

        Hide
        Timothy Bish added a comment -

        Pretty sure bad things will happen if the initalReconnectDelay is set to zero using this patch.

        Show
        Timothy Bish added a comment - Pretty sure bad things will happen if the initalReconnectDelay is set to zero using this patch.
        Hide
        Jeff Genender added a comment -

        Ah right... 0 == forever... ok... fixed it with a guard. All works now.

        Show
        Jeff Genender added a comment - Ah right... 0 == forever... ok... fixed it with a guard. All works now.
        Hide
        Jeff Genender added a comment -

        AMQ-3513-take2.patch is attached with the guard.

        Show
        Jeff Genender added a comment - AMQ-3513 -take2.patch is attached with the guard.
        Hide
        Jeff Genender added a comment -

        Patch take 3... I moved the delay above the while loop, so it really is just an initial reconnect and not on every connection.

        Show
        Jeff Genender added a comment - Patch take 3... I moved the delay above the while loop, so it really is just an initial reconnect and not on every connection.
        Hide
        Timothy Bish added a comment -

        Patch applied in trunk.

        Show
        Timothy Bish added a comment - Patch applied in trunk.

          People

          • Assignee:
            Unassigned
            Reporter:
            Jeff Genender
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development