Page MenuHomePhabricator

Move repository URIs to a dedicated index
ClosedPublic

Authored by epriestley on Jan 12 2016, 3:06 PM.
Tags
None
Referenced Files
F13371687: D15005.diff
Fri, Jun 28, 4:32 AM
F13360569: D15005.diff
Tue, Jun 25, 11:36 AM
F13324354: D15005.diff
Fri, Jun 14, 11:37 PM
F13311533: D15005.diff
Tue, Jun 11, 5:09 AM
F13297593: D15005.diff
Fri, Jun 7, 5:44 AM
F13280738: D15005.diff
Sun, Jun 2, 9:14 AM
F13241980: D15005.diff
May 23 2024, 1:16 AM
F13230742: D15005.id36245.diff
May 20 2024, 11:17 PM
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.