Unwrap CompletionException automatically about ea-async HOT 4 CLOSED

This has been a subject of much debate internally at EA on whether ea-async should unpack the exception or not.

The argument for making the change is as you rightly point out, it's annoying to have to catch CompletionException and then unpack it.

The argument for returning the CompletionException as is, is that it's consistent with the JDK API, such as if you call CompletableFuture.join.

from ea-async.

Is it possible to make behaviour configurable? For example, as a java agent option

from ea-async.

In my opinion, there is one reason not to auto-unpack the CompletionException (even if it is annoying).

If you want to catch a checked exception, the compiler will complain, because as far as it knows, await doesn't throw checked exceptions. An ugly workaround would be to catch Exception and check its type using instanceof, which is hardly less verbose than unwrapping the CompletionException.

from ea-async.

Yeh @Maia-Everett. That has been one of the reasons folks have discussed internally.
In general I'm not a fan of doing the unpacking because it's inconsistent with the expected behavior of CompletableFuture.

from ea-async.