Page MenuHomeElementl

Do Postgres/MySQL table creation in a transaction
ClosedPublic

Authored by dgibson on Mon, Jun 7, 2:55 PM.

Details

Summary

User asked why we don't do this and I didn't have a great answer. Let's do the table creation and alembic writes all in one big transaction.

(There is still some jankiness from the fact that we share alembic configs between the three different storage classes - ideally there would be some way to do the full table creation all in one transaction, but this is still an improvement I think)

Test Plan

BK

Diff Detail

Repository
R1 dagster
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Mon, Jun 7, 3:28 PM
Harbormaster failed remote builds in B31751: Diff 39100!

something about doing this in a transaction now made sqlalchemy complain about not setting an index length on some indices in mysql, picked a reasonable-looking default. Do I need to add these as a migration? (The index isn't new, just hte length restriction)

This revision is now accepted and ready to land.Tue, Jun 8, 8:34 PM