HomePhabricator

Remove backwards-compatible run_config from execution context (#1874)

Authored by nate on Feb 9 2020, 7:19 PM.

Description

Remove backwards-compatible run_config from execution context (#1874)

Summary:
Started this for https://github.com/dagster-io/dagster/issues/1874 - seems like one thing to resolve is whether create_execution_plan should take a RunConfig or a PipelineRun as an argument, which I guess depends on how much we consider this a user-facing API?

There's a bit of both in this diff, would love to discuss and we can pick one and I'll follow-up here.

Test Plan: unit

Reviewers: prha

Reviewed By: prha

Differential Revision: https://dagster.phacility.com/D1986

Details

Committed
nateFeb 10 2020, 10:03 PM
Reviewer
prha
Differential Revision
D1986: Remove backwards-compatible run_config from execution context (#1874)
Parents
R1:66a9d86ec047: 2 step re-execution test
Branches
Unknown
Tags
Unknown