curator-user mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jordan Zimmerman <>
Subject Re: RetryLoop bug?
Date Fri, 13 Jun 2014 22:18:48 GMT
Though, I could see an argument against it.


From: Jordan Zimmerman
Reply: Jordan Zimmerman
Date: June 13, 2014 at 9:07:26 AM
To: Cameron McKenzie,
Subject:  Re: RetryLoop bug?  

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


From: Cameron McKenzie
Date: June 12, 2014 at 11:27:22 PM
Subject:  RetryLoop bug?

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.


View raw message