-
Notifications
You must be signed in to change notification settings - Fork 275
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Turn RetryException into RuntimeException? #2
Comments
good thing there is a mechanism called javadoc to show which exceptions are thrown even if they are not checked. |
I should probably point out that @dirkraft likes to troll me occasionally in lolspeak. He and I have actually had discussions about switching the checked |
I think the |
I like the checked exceptions too – if someone is going to the bother of using a Retrier, it's clear that they care about catching exceptions. |
How about wrapping the cause of the ExecutionException in a RetryException? (and hide it completely from API consumers) |
I, too, liked the checked exception. |
Checked exceptions leave the retry pretty much outside of the clean lamda world. Java 8 is here, and unless a Runtime Exception is used I (and probably others) will prefer other retry solutions. |
pls, I like when my exceptions explode all the way to the top liek in php. kthx
The text was updated successfully, but these errors were encountered: