HomePhabricator

Impose a HardpointEngine future parallelism limit

Description

Impose a HardpointEngine future parallelism limit

Summary:
Ref T13546. If we try to resolve several hundred hardpoint queries which execute subprocesses, we can currently hit system limits.

For now, limit resolution to 32 simultaneous futures. In the future, this should switch to FuturePool and possibly become more nuanced.

Test Plan: In a future change, ran arc land --into-empty ... to land thousands of commits. Before change, got a "proc_open()" error when launching too many simultaneous subprocesses. After change, this "worked".

Maniphest Tasks: T13546

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

Details

Provenance
epriestleyAuthored on Jun 4 2020, 4:40 PM
epriestleyPushed on Jun 5 2020, 7:15 PM
Differential Revision
D21307: Impose a HardpointEngine future parallelism limit
Parents
rARC7378e2baad69: Remove special casing of "arc --version"
Branches
Unknown
Tags
Unknown
Tasks
T13546: Modernize the "arc land" workflow
Build Status
Buildable 24556
Build 33850: Run Core Tests