Launchpad itself

Merge lp://staging/~jelmer/launchpad/branchjob-cache-authors into lp://staging/launchpad

  1. branchjob-cache-authors
  2. Merge into devel
Proposed by Jelmer Vernooij
Status: Merged
Approved by: Jelmer Vernooij
Approved revision: no longer in the source branch.
Merged at revision: 14786
Proposed branch: lp://staging/~jelmer/launchpad/branchjob-cache-authors
Merge into: lp://staging/launchpad
Diff against target: 116 lines (+47/-12)
2 files modified
lib/lp/code/model/branchjob.py (+26/-11)
lib/lp/code/model/tests/test_branchjob.py (+21/-1)
To merge this branch: bzr merge lp://staging/~jelmer/launchpad/branchjob-cache-authors
Reviewer Review Type Date Requested Status
Abel Deuring (community) code Approve
Review via email: mp+92470@code.staging.launchpad.net

Commit message

[r=adeuring][bug=930166] Cache revision authors in branchjob.

Description of the change

When processing new branches, don't repeatedly look up author display names for authors we've already seen.

This should help a bit towards fixing bug 808930.

To post a comment you must log in.
Revision history for this message
Abel Deuring (adeuring) wrote :

looks good

review: Approve (code)

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.