ActiveMQ
  1. ActiveMQ
  2. AMQ-2047

FanoutTransport does not honor the initialReconnectDelay

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 5.1.0, 5.2.0
    • Fix Version/s: 5.3.0
    • Component/s: Transport
    • Labels:
      None
    • Patch Info:
      Patch Available

      Description

      The TransportHandler does not honor the passed in "initialReconnectDelay" parameter. It has a hard-coded value of "10" milliseconds.

      Would also like to point out that the code seems to assume that the child transport has been created successfully as soon as the "TransportFactory.compositeConnect()" returns. During runtime, if a given node is down, the exception is actually thrown a little further down the code - when the "restoreTransport()" method is called.

      1. patch.txt
        1 kB
        Mohan Kishore

        Activity

        Hide
        Mohan Kishore added a comment -

        patch against 5.1 class - basically pushes the "post successful connect" code a little lower in the code - until after the "restoreTransport()" call.

        Also ensures that the "initialReconnectDelay" parameter is honored after a reconnect.

        Show
        Mohan Kishore added a comment - patch against 5.1 class - basically pushes the "post successful connect" code a little lower in the code - until after the "restoreTransport()" call. Also ensures that the "initialReconnectDelay" parameter is honored after a reconnect.
        Hide
        Dejan Bosanac added a comment -

        Committed in the SVN revision 734766. Thanks for the fix

        Show
        Dejan Bosanac added a comment - Committed in the SVN revision 734766. Thanks for the fix

          People

          • Assignee:
            Dejan Bosanac
            Reporter:
            Mohan Kishore
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development