Launchpad itself

Merge ~wgrant/launchpad:bprdc-ios into launchpad:master

  1. Git
  2. lp:~wgrant/launchpad
  3. bprdc-ios
  4. Merge into master
Proposed by William Grant
Status: Merged
Approved by: William Grant
Approved revision: 4793ee3df1c1700fa8487fd4bd0079dcd0f29cf9
Merge reported by: Otto Co-Pilot
Merged at revision: not available
Proposed branch: ~wgrant/launchpad:bprdc-ios
Merge into: launchpad:master
Diff against target: 14 lines (+8/-0)
1 file modified
database/schema/patch-2210-01-4.sql (+8/-0)
Reviewer Review Type Date Requested Status
Colin Watson (community) db Approve
Review via email: mp+379942@code.qastaging.launchpad.net

Commit message

Index BinaryPackageReleaseDownloadCount for index-only scans

Description of the change

Creating an index on binarypackagereleasedownloadcount (archive, binary_package_release, day, count) allows BinaryPackagePublishingHistory.getDailyDownloadTotals and Archive.getPackageDownloadCount to execute as index-only scans and not touch the heap at all. One sample query drops from 2789 page reads to 177.

Should be applied hot.

To post a comment you must log in.
Revision history for this message
Colin Watson (cjwatson) :
review: Approve (db)

There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.

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.

Subscribers

People subscribed via source and target branches

to status/vote changes: