diff options
author | 2018-06-18 16:16:36 -0700 | |
---|---|---|
committer | 2018-06-18 16:18:08 -0700 | |
commit | 9e91f20bb31e44b4be836996ac9d91af61e1e822 (patch) | |
tree | 7e7c116290099ef5a8e16e67a9d4f5a0c908fbf9 /src/main/java/com/google/devtools/build/lib/skyframe/WorkspaceStatusFunction.java | |
parent | d4406d644efcf86327834a56b01cc212389e52e7 (diff) |
Remove support for --discard_actions_after_execution.
The memory savings from this flag are not worth the complexity, and it interferes with action restarting.
RELNOTES: Remove support for --discard_actions_after_execution.
PiperOrigin-RevId: 201077905
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/skyframe/WorkspaceStatusFunction.java')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/skyframe/WorkspaceStatusFunction.java | 10 |
1 files changed, 1 insertions, 9 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/skyframe/WorkspaceStatusFunction.java b/src/main/java/com/google/devtools/build/lib/skyframe/WorkspaceStatusFunction.java index 68e6550c86..817de3954e 100644 --- a/src/main/java/com/google/devtools/build/lib/skyframe/WorkspaceStatusFunction.java +++ b/src/main/java/com/google/devtools/build/lib/skyframe/WorkspaceStatusFunction.java @@ -18,7 +18,6 @@ import com.google.devtools.build.lib.analysis.WorkspaceStatusAction; import com.google.devtools.build.skyframe.SkyFunction; import com.google.devtools.build.skyframe.SkyKey; import com.google.devtools.build.skyframe.SkyValue; -import java.util.function.Supplier; /** Creates the workspace status artifacts and action. */ public class WorkspaceStatusFunction implements SkyFunction { @@ -26,13 +25,10 @@ public class WorkspaceStatusFunction implements SkyFunction { WorkspaceStatusAction create(String workspaceName); } - private final Supplier<Boolean> removeActionAfterEvaluation; private final WorkspaceStatusActionFactory workspaceStatusActionFactory; WorkspaceStatusFunction( - Supplier<Boolean> removeActionAfterEvaluation, WorkspaceStatusActionFactory workspaceStatusActionFactory) { - this.removeActionAfterEvaluation = Preconditions.checkNotNull(removeActionAfterEvaluation); this.workspaceStatusActionFactory = workspaceStatusActionFactory; } @@ -49,11 +45,7 @@ public class WorkspaceStatusFunction implements SkyFunction { WorkspaceStatusAction action = workspaceStatusActionFactory.create(workspaceNameValue.getName()); - return new WorkspaceStatusValue( - action.getStableStatus(), - action.getVolatileStatus(), - action, - removeActionAfterEvaluation.get()); + return new WorkspaceStatusValue(action.getStableStatus(), action.getVolatileStatus(), action); } @Override |