HomeElementl

[RFC] pipeline contains a graph

Description

[RFC] pipeline contains a graph

Summary:
Change PipelineDefinition from being a GraphDefinition to *containing* a GraphDefinition.

I tried to avoid breaking changes here, notably

  • only changing PipelineDefinition constructor by adding graph_def
  • adding proxy methods for graph properties. Since i highly doubt users hit any of these - I could also migrate the callsites to go through .graph

Test Plan: bk

Reviewers: sandyryza, cdecarolis

Reviewed By: sandyryza, cdecarolis

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

Details

Provenance
alangenfeldAuthored on Feb 23 2021, 11:02 PM
Reviewer
sandyryza
Differential Revision
D6648: [RFC] pipeline contains a graph
Parents
R1:2d55c30b5eff: Include solid definitions in invocation docs
Branches
Unknown
Tags
Unknown