Page MenuHomeElementl
Differential D9167

Bring back "Load execution plan from the snapshot on the run when resume retrying""
ClosedPublic
Actions

Authored by • dgibson on Jul 30 2021, 7:11 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Mar 22, 12:50 AM
Unknown Object (File)
Fri, Mar 10, 11:34 PM
Unknown Object (File)
Fri, Mar 10, 11:34 PM
Unknown Object (File)
Fri, Mar 10, 11:34 PM
Unknown Object (File)
Fri, Mar 10, 11:34 PM
Unknown Object (File)
Fri, Mar 10, 11:30 PM
Unknown Object (File)
Thu, Mar 9, 2:13 PM
Unknown Object (File)
Feb 10 2023, 2:24 AM
View All 96 Files
Subscribers
None

Details

Reviewers
• prha
Commits
R1:b33469677b47: Bring back "Load execution plan from the snapshot on the run when resume…
Summary

This reverts commit bfb90e8b0b442f657e5256082d3116aefa8c330b. Added a subsetting case for the backfill logic that was previously failing - i'm ok keeping the from failure case a bit slower if it means a 'standard' backfill can do fewer gRPC calls

Test Plan

BK (new backfill test that phil added passes)

Diff Detail

Repository
R1 dagster
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
python_modules/
dagster-graphql/
dagster_graphql/
implementation/
execution/
10 lines
dagster_graphql_tests/
graphql/
2 lines
dagster/
dagster/
core/
4 lines
execution/
55 lines
plan/
24 lines

Diff 42691

View Options

python_modules/dagster-graphql/dagster_graphql/implementation/execution/run_lifecycle.py

Loading...
View Options

python_modules/dagster-graphql/dagster_graphql_tests/graphql/setup.py

Loading...
View Options

python_modules/dagster/dagster/core/errors.py

Loading...
View Options

python_modules/dagster/dagster/core/execution/backfill.py

Loading...
View Options

python_modules/dagster/dagster/core/execution/plan/resume_retry.py

Loading...