Skip to content
Snippets Groups Projects

migrator: Do not require explicit transactions in migration definitions

Merged 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

Merged by avatar (Jul 6, 2025 10:06am UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading