Page MenuHomePhabricator

Move repository URIs to a dedicated index
ClosedPublic

Authored by epriestley on Jan 12 2016, 3:06 PM.
Tags
None
Referenced Files
F18904618: D15005.id36245.diff
Sat, Nov 8, 10:09 AM
F18887418: D15005.id.diff
Fri, Nov 7, 5:59 AM
F18880709: D15005.diff
Thu, Nov 6, 8:32 PM
F18810827: D15005.id.diff
Sun, Oct 19, 11:39 PM
F18737733: D15005.id36255.diff
Oct 1 2025, 12:20 PM
F18736445: D15005.id.diff
Oct 1 2025, 5:58 AM
F18733097: D15005.diff
Sep 30 2025, 9:23 PM
F18713039: D15005.id.diff
Sep 29 2025, 5:35 AM
Subscribers
None

Details

Summary

Ref T4705 (there are also some other adjacent related tasks dealing with URIs).

Currently, we issue a "get repositories matching URIs: ..." query by loading every possible repository and then checking their URIs in PHP.

Instead, put URIs in a separate table. I plan for each repository to potentially have multiple URIs soon, so this prepares for that.

Test Plan
  • Ran migrations.
  • Looked at index table, made sure it appeared sensible.
  • Ran some queries by uri to find repositories, found the repositories I expected.
  • Updated the remote URI of a repository, saw queries / index update appropriately.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Move repository URIs to a dedicated index.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Jan 12 2016, 6:27 PM
This revision was automatically updated to reflect the committed changes.