Skip to content

Commit

Permalink
Always use RewindableGraphInconsistencyReceiver
Browse files Browse the repository at this point in the history
  • Loading branch information
illicitonion committed Apr 26, 2022
1 parent 0670bc4 commit 95c71f0
Showing 1 changed file with 1 addition and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -167,7 +167,7 @@ public final class SequencedSkyframeExecutor extends SkyframeExecutor {
private Duration outputTreeDiffCheckingDuration = Duration.ofSeconds(-1L);

private final WorkspaceInfoFromDiffReceiver workspaceInfoFromDiffReceiver;
private GraphInconsistencyReceiver inconsistencyReceiver = GraphInconsistencyReceiver.THROWING;
private GraphInconsistencyReceiver inconsistencyReceiver = new RewindableGraphInconsistencyReceiver();

private SequencedSkyframeExecutor(
Consumer<SkyframeExecutor> skyframeExecutorConsumerOnInit,
Expand Down Expand Up @@ -273,11 +273,6 @@ public WorkspaceInfoFromDiff sync(
OptionsProvider options)
throws InterruptedException, AbruptExitException {
if (evaluatorNeedsReset) {
// Rewinding is only supported with no incremental state and no action cache.
inconsistencyReceiver =
trackIncrementalState || useActionCache(options)
? GraphInconsistencyReceiver.THROWING
: new RewindableGraphInconsistencyReceiver();
// Recreate MemoizingEvaluator so that graph is recreated with correct edge-clearing status,
// or if the graph doesn't have edges, so that a fresh graph can be used.
resetEvaluator();
Expand Down

0 comments on commit 95c71f0

Please sign in to comment.