Page MenuHomePhabricator

Don't try to rebuild a repository graph cache bucket more than once per request
ClosedPublic

Authored by epriestley on May 11 2014, 3:50 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, May 7, 4:57 AM
Unknown Object (File)
Fri, May 3, 2:22 AM
Unknown Object (File)
Mon, Apr 29, 1:44 PM
Unknown Object (File)
Wed, Apr 24, 9:49 PM
Unknown Object (File)
Sun, Apr 21, 3:51 PM
Unknown Object (File)
Wed, Apr 17, 2:52 PM
Unknown Object (File)
Fri, Apr 12, 1:21 AM
Unknown Object (File)
Thu, Apr 11, 7:22 AM
Subscribers

Details

Summary

Ref T2683. This is a small optimization, but it has low complexity: don't rebuild a bucket more than once in the same request, since it will almost always be the same. Bucket rebuilds are pretty cheap, but this saves a few queries.

Test Plan
  • After discovering (but before parsing) a commit, viewed its browse view. Verified that this patch causes us to perform only one bucket rebuild, and therefore reduces the number of queries we issue.
  • Parsed the commit and viewed the browse view again, got successful rebuild and then fills from cache.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Don't try to rebuild a repository graph cache bucket more than once per request.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.May 12 2014, 6:30 PM
epriestley updated this revision to Diff 21575.

Closed by commit rP84fc8f0baf9f (authored by @epriestley).