Retrying is correct. Failure to connect is in the class of recoverable 
exceptions so it’s appropriate to retry based on the configured policy. 

-Jordan


From: Cameron McKenzie [email protected]
Reply: [email protected] [email protected]
Date: June 12, 2014 at 11:27:22 PM
To: [email protected] [email protected]
Subject:  RetryLoop bug?  

Guys,
I was poking through the RetryLoop code while looking at an unrelated bug and 
noticed that before doing a retry it calls 
internalBlockUntilConnectedOrTimedOut(). This makes sense. There's no point in 
doing a retry if you're not connected. The thing is that it doesn't really 
achieve anything because the retry still occurs even if a connection is not 
established.

Is this intentional? Doesn't seem to make a lot of sense to me. It would make 
more sense to call the blockUntilConnectedOrTimedOut() method, and if that 
returns false, then just treat this as a failed retry.

Thoughts?
cheers
Cam


Reply via email to