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

1 files changed, 0 insertions, 5 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/exec/FileWriteStrategy.java b/src/main/java/com/google/devtools/build/lib/exec/FileWriteStrategy.java
index 9c476bc648..1359363607 100644
--- a/src/main/java/com/google/devtools/build/lib/exec/FileWriteStrategy.java
+++ b/src/main/java/com/google/devtools/build/lib/exec/FileWriteStrategy.java
@@ -71,9 +71,4 @@ public final class FileWriteStrategy implements FileWriteActionContext {
   public ResourceSet estimateResourceConsumption(AbstractFileWriteAction action) {
     return action.estimateResourceConsumptionLocal();
   }
-
-  @Override
-  public String strategyLocality(AbstractFileWriteAction action) {
-    return "local";
-  }
 }