Don't pass temporary direct deps to environment when bubbling up.
Normally, any known direct deps are guaranteed to be done. That's not true when bubbling. Since knowing no direct deps is a plausible state of affairs, just don't pass any. This may aid future code changes.
--
MOS_MIGRATED_REVID=101980323
diff --git a/src/main/java/com/google/devtools/build/skyframe/ParallelEvaluator.java b/src/main/java/com/google/devtools/build/skyframe/ParallelEvaluator.java
index 6ddd647..88bd0ee 100644
--- a/src/main/java/com/google/devtools/build/skyframe/ParallelEvaluator.java
+++ b/src/main/java/com/google/devtools/build/skyframe/ParallelEvaluator.java
@@ -1248,8 +1248,7 @@
}
}
SkyFunctionEnvironment env =
- new SkyFunctionEnvironment(parent, parentEntry.getTemporaryDirectDeps(),
- bubbleErrorInfo, visitor);
+ new SkyFunctionEnvironment(parent, ImmutableSet.<SkyKey>of(), bubbleErrorInfo, visitor);
externalInterrupt = externalInterrupt || Thread.currentThread().isInterrupted();
try {
// This build is only to check if the parent node can give us a better error. We don't
