Page MenuHomePhabricator

Add a unique key to OwnersPath on "<packageID, repositoryPHID, pathIndex>"
ClosedPublic

Authored by epriestley on Mar 7 2018, 3:47 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, May 7, 7:48 AM
Unknown Object (File)
Fri, May 3, 7:27 AM
Unknown Object (File)
Mon, Apr 29, 8:42 AM
Unknown Object (File)
Sun, Apr 28, 11:59 AM
Unknown Object (File)
Thu, Apr 25, 1:33 AM
Unknown Object (File)
Fri, Apr 19, 7:31 PM
Unknown Object (File)
Thu, Apr 11, 9:08 AM
Unknown Object (File)
Mar 31 2024, 4:53 AM
Subscribers
None

Details

Summary

Depends on D19181. Ref T11015. This nukes duplicates from the table if they exist, then adds a unique key.

(Duplicates should not exist and can not be added with any recent version of the web UI.)

Test Plan
  • Tried to add duplicates with web UI, didn't have any luck.
  • Explicitly added duplicates with manual INSERTs.
  • Viewed packages in web UI and saw duplicates.
  • Ran migrations, got a clean purge and a nice unique key.
  • There's still no way to actually hit a duplicate key error in the UI (unless you can collide hashes, I suppose), this is purely a correctness/robustness change.

Diff Detail

Repository
rP Phabricator
Branch
opath2
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 19757
Build 26762: Run Core Tests
Build 26761: arc lint + arc unit