[QPID-895] FailoverSingleServer delays on first connection.

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: M2.1, M3
• Fix Version/s: M3
• Component/s: JMS AMQP 0-x
• Assignee: Martin Ritchie
• Reporter: Martin Ritchie
• Created: 02/Apr/08 09:43
• Updated: 14/Mar/17 20:02
• Resolved: 23/Jul/08 14:54

Description

Summary:
_currentRetries is -1 on first connection but at the delay point in getNextBroker it has already been incremented.

I'm sure the code was supposed to incorrectly say '-1' rather than '1' but both of those are wrong. We want to delay on all subsequent connection attempts which means '> 0'.

• if (delayStr != null && _currentRetries != 1)
  + if (delayStr != null && _currentRetries > 0)

So this should fix the issue. A work around is to use two brokers in the URL with their own connectdelay that way it will use the RoundRobin Method which doesn't have this bug.

Issue Links

• relates to: QPID-690 FailoverHandler does not have a delay between reconnection attempts.