← Back to team overview

launchpad-reviewers team mailing list archive

Re: [Merge] lp:~rvb/launchpad/activereviews-bug-867941 into lp:launchpad

 

Something weird is going on here; if I change:

recorder1 = self.createBMPsAndMeasureActiveReviewsPageRendering(3)
recorder2 = self.createBMPsAndMeasureActiveReviewsPageRendering(7)

to:

recorder1 = self.createBMPsAndMeasureActiveReviewsPageRendering(3)
recorder2 = self.createBMPsAndMeasureActiveReviewsPageRendering(70)

Then the number of queries does not match.  The extra queries are the one needed to get branch.{_associatedProductSeries, _associatedSuiteSourcePackages} which is exactly what _preloadDataForBranches populates.

This needs investigation…
-- 
https://code.launchpad.net/~rvb/launchpad/activereviews-bug-867941/+merge/82375
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~rvb/launchpad/activereviews-bug-867941 into lp:launchpad.


References