diff options
| author |  Philipp Wollermann <philwo@google.com> | 2016-02-03 13:18:26 +0000 |
|---|---|---|
| committer |  David Chen <dzc@google.com> | 2016-02-03 22:06:30 +0000 |
| commit | 9446e9b55e01aee0425a1f21b47b98df55043c6f (patch) | |
| tree | 59a0bdbfa93b133ed5c53ec41de9af165319a927 /src/test/java/com/google/devtools/build/lib/actions/util/TestAction.java | |
| parent | 0d1a53cf5614d621518914682ec44c6f20a27ab7 (diff) | |
Make The Build Faster: Drop the describeStrategy() and strategyLocality() methods, as we can simply pass an ActionStatusMessage to the EventBus instead. All SpawnActionContexts now send an appropriate message when they execute a Spawn.
This also gets rid of the idiom that an Action knows which strategy will be used to execute it - this decision and knowledge belongs to the executor, not the action.
--
MOS_MIGRATED_REVID=113731846
Diffstat (limited to 'src/test/java/com/google/devtools/build/lib/actions/util/TestAction.java')
| -rw-r--r-- | src/test/java/com/google/devtools/build/lib/actions/util/TestAction.java | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/src/test/java/com/google/devtools/build/lib/actions/util/TestAction.java b/src/test/java/com/google/devtools/build/lib/actions/util/TestAction.java index 61f092f6cf..4b5485dba4 100644 --- a/src/test/java/com/google/devtools/build/lib/actions/util/TestAction.java +++ b/src/test/java/com/google/devtools/build/lib/actions/util/TestAction.java @@ -128,11 +128,6 @@ public class TestAction extends AbstractAction { } @Override - public String describeStrategy(Executor executor) { - return ""; - } - - @Override protected String computeKey() { List<String> outputsList = new ArrayList<>(); for (Artifact output : getOutputs()) { |