Page MenuHomePhabricator

Move repository URIs to a dedicated index
ClosedPublic

Authored by epriestley on Jan 12 2016, 3:06 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Dec 12, 5:36 AM
Unknown Object (File)
Wed, Nov 27, 11:35 PM
Unknown Object (File)
Wed, Nov 27, 1:14 AM
Unknown Object (File)
Sat, Nov 23, 5:31 AM
Unknown Object (File)
Sat, Nov 23, 4:31 AM
Unknown Object (File)
Mon, Nov 18, 6:31 PM
Unknown Object (File)
Nov 15 2024, 3:08 PM
Unknown Object (File)
Nov 11 2024, 5:54 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.