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)
Fri, Sep 6, 4:50 AM
Unknown Object (File)
Wed, Sep 4, 7:57 PM
Unknown Object (File)
Mon, Sep 2, 1:15 AM
Unknown Object (File)
Aug 28 2024, 3:28 AM
Unknown Object (File)
Aug 22 2024, 3:57 AM
Unknown Object (File)
Aug 17 2024, 8:11 AM
Unknown Object (File)
Aug 17 2024, 12:07 AM
Unknown Object (File)
Aug 9 2024, 3:50 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.