Clone
Paul Merlin
committed
on 06 Apr 17
Fix TaskPlanExecutor usage of worker leases
ParallelTaskPlanExecutor, using a thread-pool, was taking worker leases
from the forked threads … Show more
Fix TaskPlanExecutor usage of worker leases

ParallelTaskPlanExecutor, using a thread-pool, was taking worker leases

from the forked threads without passing its current worker lease. The

child operations could then not borrow their parent lease. This consumed

one spurious lease.

This, combined with parallel download of artifacts, composite builds

and indirect cyclic dependencies across composites made the

CompositeBuildDependencyCycleIntegrationTest hang.

This change fix [Default|Parallel]TaskPlanExecutor to do the right thing

Show less

master + 414 more