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 14:07:09 GMT
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