Differential D5313

Push loading behavior to StepInputSource and make it a thunk and add versioning and required resource keys to source
Needs ReviewPublic
Actions

Authored by schrockn on Nov 28 2020, 6:21 PM.

Tags

None

Subscribers

None

This revision needs review, but there are no reviewers specified.

Details

Reviewers: None

Summary

TODO

Test Plan

BK

Diff Detail

Repository

Branch

attempt-to-refactor-thunk-style-all-combined

Lint

Lint OK

Unit

No Unit Test Coverage

Event Timeline

schrockn created this revision.Nov 28 2020, 6:21 PM

Harbormaster completed remote builds in B21826: Diff 26509.Nov 28 2020, 6:41 PM

schrockn requested review of this revision.Nov 28 2020, 6:41 PM

Revision Contents
Changeset List

Path

Size

python_modules/

dagster/

dagster/

core/

execution/

plan/

execute_step.py

93 lines

161 lines

18 lines

resolve_versions.py

86 lines

resources_init.py

14 lines

Diff 26509

python_modules/dagster/dagster/core/execution/plan/execute_step.py

Loading...

python_modules/dagster/dagster/core/execution/plan/inputs.py

Loading...

python_modules/dagster/dagster/core/execution/plan/plan.py

Loading...

python_modules/dagster/dagster/core/execution/resolve_versions.py

Loading...

python_modules/dagster/dagster/core/execution/resources_init.py

Loading...