Replace with SolidDefSnapshot, someday this will be threaded down from above instead of created inline
Details
Details
Diff Detail
Diff Detail
- Repository
- R1 dagster
- Branch
- rmpipelinedef7
- Lint
Lint OK - Unit
No Unit Test Coverage
Event Timeline
| python_modules/dagster/dagster/core/execution/plan/compute.py | ||
|---|---|---|
| 39–70 | these certainly are not memoized - would be good to not land the stack til something on top threads the pipeline snap in to ExecutionPlan building so its available here | |
| python_modules/dagster/dagster/core/execution/plan/step.py | ||
| 75–76 | nit: its a little weird to have methods that are only usable in user mode. Having to pass in a pipeline_def makes it pretty clear, but it would be cool if there was a cleaner way to model it | |