diff options
author | Ulf Adams <ulfjack@google.com> | 2017-03-02 17:32:28 +0000 |
---|---|---|
committer | Yue Gan <yueg@google.com> | 2017-03-03 10:36:39 +0000 |
commit | cddaaa66b9aac7c4ea4772ae2a23b97426d267c2 (patch) | |
tree | eb018b744eff97144ccc2fb3850dfc184b907236 /src/main/java/com/google/devtools/build/lib/sandbox/BUILD | |
parent | 64f3eb0492f675d0e4713989f2bdd175ff078758 (diff) |
Rationalize local resource acquisition
Move all local resource acquisition to where local execution actually happens.
Don't attempt to acquire resources per action, but only for individual spawns.
This significantly simplifies the code.
The downside is that we don't account for action-level work anymore. In
general, actions should not perform any process execution themselves, but
always delegate such work to a SpawnStrategy implementation.
This change makes sure that every Spawn has local resources set in a way that
is consistent with the previous state.
However, there are two actions - Fileset and FileWrite -, which are not spawns,
and so we now don't limit their concurrent execution anymore. For Fileset, all
work is done in a custom Fileset-specific thread pool, so this shouldn't be a
problem. I'm not sure about FileWriteAction.
--
PiperOrigin-RevId: 149012600
MOS_MIGRATED_REVID=149012600
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/sandbox/BUILD')
0 files changed, 0 insertions, 0 deletions