HomePhabricator

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

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

Provenance
epriestleyAuthored on Feb 26 2020, 4:35 PM
epriestleyPushed on Mar 29 2020, 5:14 PM
Differential Revision
D21031: Remove the "timeout" parameter from "Future->resolve()"
Parents
rARC33b9728b5f65: Run ls-files from the root of the directory
Branches
Unknown
Tags
Unknown
Tasks
T11968: Decide the fate of FutureGraph
Build Status
Buildable 23957
Build 32968: Run Core Tests