Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure that's what that says. It suggests resetting the interrupted flag if you don;t handle the exception, but here it's handled by throwing an IOException.

And now that I look at this, is that even right? We're sleeping between retries, the sleep is interrupted, so we cancel all the retries and throw an IOException instead of returning. I guess that makes sense, but at this point the interrupt has been handled so it doesn't seem like the current thread should still have its interrupt flag set.