• migration script
• remove backfill script, add tests

• update tests

• remove backfill script, add tests

I added a migration script to the cli using the command dagster instance backfill event_log.

Add migration script to CLI

Should note that step_key in event_log is a nullable field that is only used for longitudinal views. Nothing should break, but step_key based views will just only have data going forward from this point.

updated

updates

updates

still need to rebuild the snapshots though...

these changes look good! setting the focus definitely makes the transition less jarring.

I thought about this, but wasn't exactly sure what our attitude towards data migrations were. Talking this over yesterday with @schrockn, but one concern is the amount of time required to migrate the event_log table, just in terms of the magnitude of number of rows.

Actually, probably don't need to see this again, but please add the comment with the GH issue.

update

overall looks good.

yup, this makes sense to me...

add usage error