Page MenuHomePhabricator

Index all repository URIs, not just the "primary" repository URI
ClosedPublic

Authored by epriestley on May 10 2016, 1:27 PM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 15 2024, 8:54 AM
Unknown Object (File)
Feb 14 2024, 8:24 AM
Unknown Object (File)
Feb 1 2024, 7:11 PM
Unknown Object (File)
Jan 27 2024, 12:55 AM
Unknown Object (File)
Dec 27 2023, 6:53 PM
Unknown Object (File)
Dec 27 2023, 6:59 AM
Unknown Object (File)
Dec 22 2023, 10:32 PM
Unknown Object (File)
Dec 22 2023, 2:04 AM
Subscribers
None

Details

Summary

Ref T10923. When regenerating the URI index for a repository, index every URI.

  • Also, make the index slightly stricter (domain + path instead of just path). Excluding the domain made more sense when we were generating only first-party URIs.
  • Make the index smarter about /diffusion/123/ URIs.
  • Show normalized URIs in diffusion.repository.search results.
Test Plan
  • Ran migration.
  • Verified sensible-looking results in database.
  • Searched for a repository URI by first-party clone URI.
  • Searched for a repository URI by mirror URI.
  • Used diffusion.repository.search to get information about repository URIs.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Index all repository URIs, not just the "primary" repository URI.
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.May 10 2016, 8:54 PM
This revision was automatically updated to reflect the committed changes.