Page MenuHomePhabricator

`key_position` on CursorPosition needs to be guaranteed by migrations
Closed, ResolvedPublic

Description

Note to myself: T11823 needs to add the key_position key as part of migrations, not via adjust, since it's unique and critical. I pushed secure without adjusting (since the push wasn't entirely clean) and got us in a spot of bother. This is non-harmful and easy to resolve, just needs cleanup.

Revisions and Commits

Restricted Differential Revision

Event Timeline

epriestley added a revision: Restricted Differential Revision.Sep 18 2017, 6:52 PM
epriestley added a commit: Restricted Diffusion Commit.Sep 18 2017, 9:00 PM

I deployed this to secure without issue, so it looks like LOCK TABLES ... is promising as a general tool for migrations.