Page MenuHomePhabricator

fix fan-in with asset stores
ClosedPublic

Authored by sandyryza on Nov 27 2020, 8:07 PM.

Diff Detail

Repository
R1 dagster
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Oh god I was doing prototyping in this area without bringing in alex's refactor

This revision is now accepted and ready to land.Nov 27 2020, 8:41 PM
python_modules/dagster/dagster/core/execution/plan/execute_step.py
568–574

this pre-exists you, but this codepath still terrifies me. whereever we end up with loading/asset store/etc i would love to have a tighter contract here

This revision was automatically updated to reflect the committed changes.