Page MenuHomePhabricator

Delete container-based loading for now
ClosedPublic

Authored by schrockn on Thu, May 21, 3:40 PM.

Details

Summary

Given that it isn't under test, we are going to delete this
for now. Once our host-user process restructuring is done, we can
readd this.

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

schrockn created this revision.Thu, May 21, 3:40 PM
schrockn updated this revision to Diff 14403.Thu, May 21, 3:52 PM
schrockn retitled this revision from Comment out container things to RFC: Comment out container things.
schrockn edited the summary of this revision. (Show Details)
schrockn added reviewers: max, alangenfeld.

up

The idea is that we would readd this once all the virtualenv-based stuff is working. For now this just adds overhead to this re-architecture project

schrockn requested review of this revision.Thu, May 21, 4:06 PM
schrockn updated this revision to Diff 14413.Thu, May 21, 4:58 PM
schrockn retitled this revision from RFC: Comment out container things to Delete container-based loading for now.
schrockn edited the summary of this revision. (Show Details)
schrockn removed a reviewer: sashank.

up

max accepted this revision.Thu, May 21, 5:12 PM
This revision is now accepted and ready to land.Thu, May 21, 5:12 PM
This revision was automatically updated to reflect the committed changes.