Page MenuHomePhabricator

(snap-backed-graphql-10) DauphinPipeline no longer requires PipelineDefinition
AbandonedPublic

Authored by schrockn on Mar 22 2020, 12:41 PM.

Details

Summary

DauphinPipeline is now backed by the snapshots, meaning that it
can be backed by a serialized format. From this we should be able to
display historical pipeline structures.

Depends on D2315

Test Plan

BK

Diff Detail

Repository
R1 dagster
Branch
snap-backed-graphql-10
Lint
Lint OK
Unit
No Unit Test Coverage

Event Timeline

schrockn created this revision.Mar 22 2020, 12:41 PM
alangenfeld requested changes to this revision.Mar 23 2020, 8:22 PM

while this was a good milestone to chase - its unclear whether its "correct" since things like preset are available on a "live" pipeline but not one thats purely backed by snapshot. Maybe this is best respun as a new PipelineReference implementor that is only backed by snapshot

This revision now requires changes to proceed.Mar 23 2020, 8:22 PM
schrockn abandoned this revision.Mar 25 2020, 5:12 PM