commit eff223e0f5232558dc134706bc5bd5d405b9bd19
author ulfjack <ulfjack@google.com> Mon Jul 17 13:18:48 2017 +0200
committer Klaus Aehlig <aehlig@google.com> Mon Jul 17 13:42:17 2017 +0200
tree 30946b4b4a05596a191b227cb7015817507a735e
parent 5752463ece84ebb4fb074888cba57412ab8d86b3

Extend the SpawnRunner API

- add an id for logging; this allows us to correlate log entries for the same
  spawn from multiple spawn runner implementations in the future
- add a prefetch method to the SpawnExecutionPolicy; better than relying on
  the ActionInputPrefetcher being injected in the constructor
- add a name parameter to the report method; this is in preparation for a
  single unified SpawnStrategy implementation - it's basically the last bit of
  difference between SandboxStrategy and RemoteSpawnStrategy; they're otherwise
  equivalent (if not identical)

PiperOrigin-RevId: 162194684

src/test/java/com/google/devtools/build/lib/remote/RemoteSpawnRunnerTest.java

diff --git a/src/test/java/com/google/devtools/build/lib/remote/RemoteSpawnRunnerTest.java b/src/test/java/com/google/devtools/build/lib/remote/RemoteSpawnRunnerTest.java
index 0390994..a6d0be8 100644
--- a/src/test/java/com/google/devtools/build/lib/remote/RemoteSpawnRunnerTest.java
+++ b/src/test/java/com/google/devtools/build/lib/remote/RemoteSpawnRunnerTest.java
@@ -186,6 +186,16 @@
     }
 
     @Override
+    public int getId() {
+      return 0;
+    }
+
+    @Override
+    public void prefetchInputs(Iterable<ActionInput> inputs) throws IOException {
+      throw new UnsupportedOperationException();
+    }
+
+    @Override
     public void lockOutputFiles() throws InterruptedException {
       throw new UnsupportedOperationException();
     }
@@ -217,7 +227,7 @@
     }
 
     @Override
-    public void report(ProgressStatus state) {
+    public void report(ProgressStatus state, String name) {
       assertThat(state).isEqualTo(ProgressStatus.EXECUTING);
     }
   }