Page MenuHomePhabricator

Remove __init__ methods in EventLogStorage and RunStorage abcs

Authored by schrockn on Oct 3 2019, 9:00 PM.



Was causing ConfigurableClass mixins not to be called.

Test Plan

Load dagit on examples

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

schrockn created this revision.Oct 3 2019, 9:00 PM
alangenfeld accepted this revision.Oct 3 2019, 9:02 PM

if it passes CI lets land this but we should think through this whole set up - worth looking quick at the postgres one that the diff that added this stuff has

This revision is now accepted and ready to land.Oct 3 2019, 9:02 PM
schrockn edited the summary of this revision. (Show Details)Oct 3 2019, 9:02 PM

yeah i started an issue to track