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/main/java/com/google/devtools/build/lib/rules/test/TestRunnerAction.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/main/java/com/google/devtools/build/lib/rules/test/TestRunnerAction.java')
| -rw-r--r-- | src/main/java/com/google/devtools/build/lib/rules/test/TestRunnerAction.java | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/test/TestRunnerAction.java b/src/main/java/com/google/devtools/build/lib/rules/test/TestRunnerAction.java index 57ddcf9c56..9f09c90319 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/test/TestRunnerAction.java +++ b/src/main/java/com/google/devtools/build/lib/rules/test/TestRunnerAction.java @@ -305,11 +305,6 @@ public class TestRunnerAction extends AbstractAction implements NotifyOnActionCa return "Testing " + getTestName(); } - @Override - public String describeStrategy(Executor executor) { - return executor.getContext(TestActionContext.class).strategyLocality(this); - } - /** * Deletes <b>all</b> possible test outputs. * |