[Memoization 9/n] Add version_strategy as an arg to pipeline definition.
ClosedPublic
Actions

Authored by cdecarolis on Aug 9 2021, 10:54 PM.

Details

Reviewers

sandyryza
alangenfeld

Commits

R1:e9074463625c: [Memoization 9/n] Add version_strategy as an arg to pipeline definition.

Summary

Considering we're keeping backcompat alive for a long time, and this is super low-cost, makes sense to support it.

Test Plan

added test with using version_strategy on pipeline decorator.

Diff Detail

Repository

R1 dagster

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

cdecarolis created this revision.Aug 9 2021, 10:54 PM

Harbormaster returned this revision to the author for changes because remote builds failed.Aug 9 2021, 11:16 PM

Harbormaster failed remote builds in B34633: Diff 42844!

Harbormaster completed remote builds in B34635: Diff 42846.Aug 10 2021, 12:28 AM

cdecarolis requested review of this revision.Aug 10 2021, 12:28 AM

sandyryza accepted this revision.Aug 10 2021, 4:16 PM

This revision is now accepted and ready to land.Aug 10 2021, 4:16 PM

Closed by commit R1:e9074463625c: [Memoization 9/n] Add version_strategy as an arg to pipeline definition. (authored by Chris DeCarolis <cdguitar817@gmail.com>). · Explain WhyAug 10 2021, 5:39 PM

This revision was automatically updated to reflect the committed changes.

Chris DeCarolis <cdguitar817@gmail.com> added a commit: R1:e9074463625c: [Memoization 9/n] Add version_strategy as an arg to pipeline definition..

Revision Contents
Changeset List

Path

Size

python_modules/

dagster/

core/

definitions/

decorators/

pipeline.py

10 lines

dagster_tests/

core_tests/

test_resolve_versions.py

32 lines

Diff 42850

View Options

python_modules/dagster/dagster/core/definitions/decorators/pipeline.py