commit fd6cdaffc851c439930f38e765ade0a51fc0a872
author Googler <noreply@google.com> Thu Mar 26 08:52:59 2020 -0700
committer Copybara-Service <copybara-worker@google.com> Thu Mar 26 08:55:00 2020 -0700
tree 957829e77d32c02823344afff0cf5f1ee231c6ad
parent 366b7b1bc34f4cd3b950fc510ba1b015d6d77986

Fix wall-time of the SpawnResult in WorkerSpawnRunner

Previously we would do `currentTime - startTime`, but `Duration.between`
expects the first parameter to be the start-time and will return
negative `Duration` otherwise.

RELNOTES: Fix wall-time of the SpawnResult in WorkerSpawnRunner
PiperOrigin-RevId: 303120753

src/main/java/com/google/devtools/build/lib/worker/WorkerSpawnRunner.java

diff --git a/src/main/java/com/google/devtools/build/lib/worker/WorkerSpawnRunner.java b/src/main/java/com/google/devtools/build/lib/worker/WorkerSpawnRunner.java
index 6eaebb8..ea2b558 100644
--- a/src/main/java/com/google/devtools/build/lib/worker/WorkerSpawnRunner.java
+++ b/src/main/java/com/google/devtools/build/lib/worker/WorkerSpawnRunner.java
@@ -195,7 +195,7 @@
     Instant startTime = Instant.now();
     WorkResponse response =
         execInWorker(spawn, key, context, inputFiles, outputs, flagFiles, inputFileCache);
-    Duration wallTime = Duration.between(Instant.now(), startTime);
+    Duration wallTime = Duration.between(startTime, Instant.now());
 
     FileOutErr outErr = context.getFileOutErr();
     response.getOutputBytes().writeTo(outErr.getErrorStream());