Page MenuHomePhabricator

get_output_manager -> get_io_manager

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



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


Diff Detail

R1 dagster
Automatic diff as part of commit; lint not applicable.
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.