launchpad-reviewers team mailing list archive

Thread
Date

[Merge] lp:~stevenk/launchpad/no-more-o-n-queries-bug-dupes into lp:launchpad

To: mp+129355@xxxxxxxxxxxxxxxxxx,William Grant <me@xxxxxxxxxxxxxxxxxx>
From: Steve Kowalik <stevenk@xxxxxxxxxx>
Date: Fri, 12 Oct 2012 11:27:46 -0000
In-reply-to: <20121012062921.2986.82973.launchpad@ackee.canonical.com>
Reply-to: mp+129355@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

The proposal to merge lp:~stevenk/launchpad/no-more-o-n-queries-bug-dupes into lp:launchpad has been updated.

Commit Message changed to:

Rewrite the heavy-lifting queries in IBug.getSubscriptionForPerson() and IPersonSubscriptionInfo._getDirectAndDuplicateSubscriptions() to be performant using two CTEs, and destroy the cause of excessive queries in the second function by bulk loading.

For more details, see:
https://code.launchpad.net/~stevenk/launchpad/no-more-o-n-queries-bug-dupes/+merge/129355
-- 
https://code.launchpad.net/~stevenk/launchpad/no-more-o-n-queries-bug-dupes/+merge/129355
Your team Launchpad code reviewers is subscribed to branch lp:launchpad.

References

[Merge] lp:~stevenk/launchpad/no-more-o-n-queries-bug-dupes into lp:launchpad
From: Steve Kowalik, 2012-10-12