Page MenuHomePhabricator

1/ Modernize airline_demo setup
ClosedPublic

Authored by max on Jan 13 2021, 7:10 PM.

Details

Summary

This moves the airline demo to use the new io_manager abstractions.

Test Plan

Unit

Diff Detail

Repository
R1 dagster
Branch
arcpatch-D5963
Lint
Lint OK
Unit
No Unit Test Coverage

Event Timeline

max requested review of this revision.Jan 13 2021, 7:33 PM

Looks good! Left a comment that I think is worth addressing before merging.

examples/airline_demo/airline_demo/pipelines.py
86–87

You should be able to take out these intermediate_storage_def arguments

This revision is now accepted and ready to land.Jan 13 2021, 7:41 PM
max marked an inline comment as done.Jan 13 2021, 7:45 PM
This revision was automatically updated to reflect the committed changes.