Page MenuHomeElementl

DynamicOutput enforce constraints
ClosedPublic

Authored by alangenfeld on Dec 15 2020, 4:52 AM.

Details

Summary

ensure that the current limitations of dynamic outputs are enforced with clear error messages.

Test Plan

added tests

Diff Detail

Repository
R1 dagster
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

cool. plz consider final comments

python_modules/dagster/dagster/core/definitions/dependency.py
458–478

slightly goofy for this to be an inner function given that you aren't capturing anything via closure. (I also thought you were somehow calling it recursively because hard to tell scope with python whitespace ish)

464–466

I'm having trouble parsing this error message

467–474

Not looking forward to explaining the subtleties of this distinction

475

output

475

mild preference for "cannot" rather than "can not"

476

probably should be a 2nd sentence or "as it is downstream"

This revision is now accepted and ready to land.Dec 15 2020, 5:14 PM
python_modules/dagster/dagster_tests/core_tests/mapping_tests/test_not_allowed.py
110–117 ↗(On Diff #27832)

[2]

This revision was automatically updated to reflect the committed changes.