This avoids a CF allocation on data fetcher in the main case where there is no exception

[bbakerman]

No description provided.

[dfa1]

@bbakerman what about using a single .handle?

        return fetchedValue
                .handle((result, exception) -> {
                    fetchCtx.onCompleted(result, exception);
                    if (exception != null) {
                        result = handleFetchingException(executionContext, dataFetchingEnvironment.get(), exception);
                    }
                    return unboxPossibleDataFetcherResult(executionContext, parameters, result);
                });

[bbakerman]

What about using a single .handle?
Ahh I tried it - it wont work

The error handler returns CF<U> while .handle() needs to a <U> so we need to compose afterwards - I dont think there is any saving to be made here

[bbakerman]

never used

[bbakerman]

never used

[bbakerman]

technically a breaking change BUT a low risk one