HomePhabricator

Remove the "timeout" parameter from "Future->resolve()"

Authored by epriestley on Feb 26 2020, 4:35 PM.

Description

Remove the "timeout" parameter from "Future->resolve()"

Summary:
Ref T11968. This future-level parameter has no nontrivial callers and makes the "fate of FutureGraph" changes more difficult.

Callers that are genuinely interested in this behavior can wrap the Future in a FutureIterator and use "setUpdateInterval()" to get the same behavior.

Test Plan: Grepped for "resolve()" and "resolvex()", updated callers.

Maniphest Tasks: T11968

Differential Revision: https://secure.phabricator.com/D21031

Details

Committed
epriestleyMar 29 2020, 5:13 PM
Parents
rARC33b9728b5f65: Run ls-files from the root of the directory
Branches
Unknown
Tags
Unknown
Tasks
T11968: Reduce use of global and public variables
ChangeId
None