Page MenuHomePhabricator

Make discovery slightly cheaper in the common case
ClosedPublic

Authored by epriestley on Apr 15 2014, 11:23 PM.
Tags
None
Referenced Files
F11039282: D8781.diff
Tue, Aug 16, 3:12 AM
F11037107: D8781.id20839.diff
Mon, Aug 15, 1:17 PM
F11036671: D8781.diff
Mon, Aug 15, 10:53 AM
Unknown Object (File)
Thu, Aug 11, 11:52 PM
Unknown Object (File)
Mon, Jul 25, 3:57 PM
Unknown Object (File)
Sun, Jul 24, 11:10 PM
Unknown Object (File)
Jul 9 2022, 10:42 PM
Unknown Object (File)
Jun 23 2022, 7:40 AM
Subscribers

Details

Summary

Ref T4605. Before discovering branches, try to prefill the cache in bulk. For repositories with large numbers of branches, this allows us to issue dramatically fewer queries.

(Before D8780, this cache was usually held across discovery events, so being able to fill it cheaply was not as relevant.)

Test Plan

Ran discovery on Git, Mercurial and SVN repositories. Observed fewer queries for Git/Mercurial.

Diff Detail

Repository
rP Phabricator
Branch
pull2
Lint
Lint Passed
Unit
Tests Passed