Page MenuHomeElementl

[5/n] Wait to determine step compute_fn until you're actually in step execution
AbandonedPublic

Authored by dgibson on Jan 21 2021, 5:46 PM.

Details

Reviewers
alangenfeld
Summary

Part N of many of making ExecutionPlan create-able within a host process.

Test Plan

BK

Diff Detail

Repository
R1 dagster
Branch
rmpipelinedef5
Lint
Lint Passed
Unit
No Test Coverage

Event Timeline

dgibson retitled this revision from Wait to determine step compute_fn until you're actually in step execution to [5/n] Wait to determine step compute_fn until you're actually in step execution.Jan 22 2021, 3:37 PM
alangenfeld added inline comments.
python_modules/dagster/dagster/core/execution/plan/execute_step.py
444–453

think we can drop the gen and None check since this func is always a generator

This revision is now accepted and ready to land.Jan 22 2021, 4:46 PM