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/analysis/util | |
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/analysis/util')
-rw-r--r-- | src/test/java/com/google/devtools/build/lib/analysis/util/AnalysisTestUtil.java | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/src/test/java/com/google/devtools/build/lib/analysis/util/AnalysisTestUtil.java b/src/test/java/com/google/devtools/build/lib/analysis/util/AnalysisTestUtil.java index 98873a2f48..f74d6e779f 100644 --- a/src/test/java/com/google/devtools/build/lib/analysis/util/AnalysisTestUtil.java +++ b/src/test/java/com/google/devtools/build/lib/analysis/util/AnalysisTestUtil.java @@ -197,11 +197,6 @@ public final class AnalysisTestUtil { } @Override - public String describeStrategy(Executor executor) { - return ""; - } - - @Override public void execute(ActionExecutionContext actionExecutionContext) throws ActionExecutionException { try { |