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
F2988819: D9167.diff
Mon, Jun 5, 4:14 AM
Unknown Object (File)
Wed, May 10, 10:32 AM
Unknown Object (File)
May 4 2023, 12:19 AM
Unknown Object (File)
May 1 2023, 3:39 AM
Unknown Object (File)
Apr 30 2023, 5:14 AM
Unknown Object (File)
Apr 30 2023, 4:24 AM
Unknown Object (File)
Apr 2 2023, 2:31 AM
Unknown Object (File)
Mar 22 2023, 12:50 AM
View All 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
Branch
unrevertphil
Lint
Lint Passed
Unit
No Test Coverage

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 42685

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...