Merge lp://qastaging/~thumper/launchpad/different-cloud into lp://qastaging/launchpad

Proposed by Tim Penhey
Status: Merged
Approved by: Robert Collins
Approved revision: no longer in the source branch.
Merged at revision: 11475
Proposed branch: lp://qastaging/~thumper/launchpad/different-cloud
Merge into: lp://qastaging/launchpad
Diff against target: 685 lines (+248/-193)
12 files modified
lib/lp/code/browser/bazaar.py (+27/-27)
lib/lp/code/configure.zcml (+1/-1)
lib/lp/code/interfaces/branch.py (+6/-3)
lib/lp/code/model/branch.py (+0/-31)
lib/lp/code/model/branchcloud.py (+52/-0)
lib/lp/code/model/revision.py (+1/-2)
lib/lp/code/model/tests/test_branchcloud.py (+65/-95)
lib/lp/code/stories/branches/xx-bazaar-home.txt (+0/-1)
lib/lp/code/stories/branches/xx-branch-tag-cloud.txt (+15/-26)
lib/lp/code/tests/helpers.py (+40/-0)
lib/lp/code/tests/test_helpers.py (+40/-0)
lib/lp/testing/tests/test_standard_test_template.py (+1/-7)
To merge this branch: bzr merge lp://qastaging/~thumper/launchpad/different-cloud
Reviewer Review Type Date Requested Status
Robert Collins (community) Approve
Review via email: mp+34146@code.qastaging.launchpad.net

Commit message

Fix the branch cloud timeouts.

Description of the change

Change what we show on the tag cloud for branches.

Instead of showing general total branch counts (hosted and mirrored), we instead show projects that have recent commits (recent being in the last 30 days and using the revision cache table).

By using the revision cache, we get *much* faster queries, and this should completely kill any time-outs we are getting for this page (crosses-fingers).

Tests:
  xx-branch-tag-cloud
  code.tests.test_helpers
  code.model.tests.test_branchcloud

To post a comment you must log in.
Revision history for this message
Robert Collins (lifeless) wrote :

doit.

review: Approve
Revision history for this message
Robert Collins (lifeless) wrote :

We may need to adjust this to normalise in some fashion but this is certainly an improvement.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.