Skip to content

migrator: Do not require explicit transactions in migration definitions

Administrator requested to merge ef/no-explicit-migration-txns into main

Created by: efritz

The explicit BEGIN and COMMIT wrapper around migration definitions were an artifact of golang-migrate. We can now make the transaction implicit only when performing migrations with no concurrent index creation. Now that we can discern the two cases, we can wrap only the one that can handle it.

Test plan

Unit tests and QA tests should give good confidence.

Merge request reports