launchpad-reviewers team mailing list archive

[Steve Kowalik <stevenk@xxxxxxxxxx>]

The proposal to merge lp:~stevenk/launchpad/builder-history-better-query into lp:launchpad has been updated.

Description changed to:

Rewrite the two (seperate) queries that are constructed by IBuildFarmJobSet.getBuildsForBuilder(). This, in conjunction with https://code.launchpad.net/~stevenk/launchpad/buildfarmjob-index/+merge/123675 pulls down the major query used on Builder:+history from 9 seconds to about 2 ms. For a further win, it reduces the line count of the function by a fair bit.

To my eyes, it looks like BuilderHistoryView already makes use of a range factory.

For more details, see:
https://code.launchpad.net/~stevenk/launchpad/builder-history-better-query/+merge/123676
-- 
https://code.launchpad.net/~stevenk/launchpad/builder-history-better-query/+merge/123676
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~stevenk/launchpad/builder-history-better-query into lp:launchpad.