Page MenuHomePhabricator

Reorganize run and event log storage
ClosedPublic

Authored by max on Fri, Nov 22, 1:41 AM.

Details

Summary

This is preparatory to a broader refactor & the introduction of alembic-based migrations.

Test Plan

Unit

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

max created this revision.Fri, Nov 22, 1:41 AM
alangenfeld accepted this revision.Fri, Nov 22, 4:31 PM

if we decided to keep in the unlikely but still breaking change lets add it to changes.md so we dont forget

python_modules/dagster/dagster/core/instance/ref.py
81

should we leave an __init__ around in storage/sqlite_run_storage in the off chance someone persisted a dagster.yaml?

This revision is now accepted and ready to land.Fri, Nov 22, 4:31 PM
max updated this revision to Diff 6792.Fri, Nov 22, 5:47 PM

Add note to changelog

This revision was landed with ongoing or failed builds.Fri, Nov 22, 5:48 PM
This revision was automatically updated to reflect the committed changes.