Page MenuHomePhabricator

object manager -> IO manager
ClosedPublic

Authored by sandyryza on Fri, Jan 8, 5:28 PM.

Details

Summary

object manager -> IO manager

Test Plan

bk, built and reviewed docs site

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.Fri, Jan 8, 5:57 PM
Harbormaster failed remote builds in B23929: Diff 29091!
Harbormaster returned this revision to the author for changes because remote builds failed.Fri, Jan 8, 7:00 PM
Harbormaster failed remote builds in B23934: Diff 29098!

can still see "object manager" and "object_manager" in comments and migration guide, e.g. object_manager_from_intermediate_storage in migration. can we grep it and make sure all are captured

@yuhan thanks for catching those ones I missed. I went through and did another pass.. I think I got everything this time.

This revision is now accepted and ready to land.Sat, Jan 9, 1:14 AM
This revision was automatically updated to reflect the committed changes.