Merge lp:~jtv/launchpad/bug-884649-branch-5 into lp:launchpad
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Julian Edwards | ||||
Approved revision: | no longer in the source branch. | ||||
Merged at revision: | 14572 | ||||
Proposed branch: | lp:~jtv/launchpad/bug-884649-branch-5 | ||||
Merge into: | lp:launchpad | ||||
Diff against target: |
292 lines (+67/-91) 2 files modified
lib/lp/archivepublisher/domination.py (+67/-78) lib/lp/archivepublisher/tests/test_dominator.py (+0/-13) |
||||
To merge this branch: | bzr merge lp:~jtv/launchpad/bug-884649-branch-5 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Julian Edwards (community) | Approve | ||
Review via email: mp+86518@code.launchpad.net |
Commit message
[r=julian-edwards][bug=884649] Use denormalized [BS]PPH.[bs]pn columns in dominator.
Description of the change
= Summary =
To get to a source package publication record's source package name, we no longer have to go through the associated source package release. There is now a column on SourcePackagePu
== Pre-implementation notes ==
The work to introduce these new columns wasn't moving along so I helped optimize the populators, kept an eye on them, retired them, added NOT NULL constraints, and initialized the sample data accordingly.
Meanwhile, on the flip side of the coin, domination is actually pretty fast already now. It probably takes less than 10% of the publication process that we're trying to speed up (though it's close enough to its current run-time target that a few percent of optimization are probably still welcome). I chose to keep thebranch because it's also a chance to make things just a bit shorter.
== Implementation details ==
Sadly, the extra column on BPPH doesn't seem to help the dominator all that much — at least not in terms of code simplicity. Most use-cases need the BinaryPackageRe
== Tests ==
{{{
./bin/test -vvc lp.archvepublisher -t dominator
}}}
== Demo and Q/A ==
Run publish-ftpmaster for Ubuntu. Domination should still work.
Similarly, run gina and verify domination for Debian.
= Launchpad lint =
Checking for conflicts and issues in changed files.
Linting changed files:
lib/lp/
lib/lp/