Page MenuHomePhabricator

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

Authored by schrockn on Sun, Mar 22, 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.Sun, Mar 22, 12:41 PM
schrockn updated this revision to Diff 10850.Sun, Mar 22, 12:46 PM
schrockn edited the summary of this revision. (Show Details)

upmessage

alangenfeld requested changes to this revision.Mon, Mar 23, 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.Mon, Mar 23, 8:22 PM
schrockn abandoned this revision.Wed, Mar 25, 5:12 PM