Replace FutureNow with JDK equivalent

Description

Presumably because the code predates Java 8, we have an implementation of a "Future that we already know the value for" in pipeline.FutureSupport. It's equivalent, I believe, to the now JDK-supplied CompletableFuture.completedFuture and we should use that instead even if it's perhaps a tiny bit slower.

Guava also has an equivalent, but let's use the JDK when we can.

Environment

None

Activity

Ian YoungOctober 18, 2022 at 1:27 PM

Done, commit 4c1eba4cf49d12527c625542e0600e53c348e966.

No new unit test needed, as the unspecified-pipeline case is already exercised.

Done

Details

Assignee

Reporter

Components

Fix versions

Affects versions

Created May 11, 2021 at 4:42 PM
Updated May 16, 2024 at 12:37 PM
Resolved October 18, 2022 at 1:31 PM

Flag notifications