Page MenuHomePhabricator

move core PipelineRun creation call
AbandonedPublic

Authored by alangenfeld on Sep 11 2019, 7:29 PM.

Details

Reviewers
None
Summary

only create pipeline runs in the API entry points where it makes sense

adds execute_run_iterator to provide a full execution API without run creation

Test Plan

buildkite

Diff Detail

Repository
R1 dagster
Branch
doin-shit
Lint
Lint OK
Unit
No Unit Test Coverage

Event Timeline

alangenfeld created this revision.Sep 11 2019, 7:29 PM
alangenfeld edited the summary of this revision. (Show Details)Sep 12 2019, 4:08 PM

seems mostly very good

python_modules/dagster/dagster/core/execution/api.py
111

function name a little weird. so only diff btwn this and execute_pipeline_iterator is that this already has a run?

alangenfeld abandoned this revision.Sep 12 2019, 9:07 PM