Cache all previously computed values in SkyQueryEnvironment#beforeEvaluateQuery if possible to save on latency for small queries.

This assumes that if the graph is up to date, then the data in SkyQueryEnvironment is also up to date. It also assumes that a ForkJoinPool remains usable until #shutdownNow is called.

--
MOS_MIGRATED_REVID=139386363

1 files changed, 2 insertions, 2 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/skyframe/PrecomputedValue.java b/src/main/java/com/google/devtools/build/lib/skyframe/PrecomputedValue.java
index d572058eac..8901745c00 100644
--- a/src/main/java/com/google/devtools/build/lib/skyframe/PrecomputedValue.java
+++ b/src/main/java/com/google/devtools/build/lib/skyframe/PrecomputedValue.java
@@ -58,7 +58,7 @@ public final class PrecomputedValue implements SkyValue {
     }
 
     void inject(Injectable injectable) {
-      injectable.inject(ImmutableMap.of(precomputed.key, new PrecomputedValue(supplier.get())));
+      injectable.inject(precomputed.key, new PrecomputedValue(supplier.get()));
     }
   }
 
@@ -179,7 +179,7 @@ public final class PrecomputedValue implements SkyValue {
      * Injects a new variable value.
      */
     public void set(Injectable injectable, T value) {
-      injectable.inject(ImmutableMap.of(key, new PrecomputedValue(value)));
+      injectable.inject(key, new PrecomputedValue(value));
     }
   }
 }