Page MenuHomePhabricator

get_output_manager -> get_io_manager
ClosedPublic

Authored by alangenfeld on Jan 21 2021, 5:31 PM.

Details

Summary

change get_output_manager to get_io_manager since its fetching the io_manager_key and expects an IOManager in a few call sites. Bumped in to this issue typing stuff in D6076

Just removed the output_manager decorator and tests since I think they don't make sense anymore, but let me know

Test Plan

bk

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

Harbormaster returned this revision to the author for changes because remote builds failed.Jan 21 2021, 5:52 PM
Harbormaster failed remote builds in B24639: Diff 30002!
This revision is now accepted and ready to land.Jan 21 2021, 6:31 PM
This revision was automatically updated to reflect the committed changes.