Merge lp:~stevenk/launchpad/db-searchablenames-for-pu into lp:launchpad/db-devel
Proposed by
Steve Kowalik
Status: | Merged |
---|---|
Merged at revision: | 12184 |
Proposed branch: | lp:~stevenk/launchpad/db-searchablenames-for-pu |
Merge into: | lp:launchpad/db-devel |
Diff against target: |
12 lines (+8/-0) 1 file modified
database/schema/patch-2209-40-0.sql (+8/-0) |
To merge this branch: | bzr merge lp:~stevenk/launchpad/db-searchablenames-for-pu |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Stuart Bishop (community) | db | Approve | |
Review via email: mp+136568@code.launchpad.net |
Description of the change
Add a searchable_names column to PackageUpload. This is supplemented by a trigram index that allows us to do efficient substring matching. This column will be set by methods on PackageUpload, such as addSource() and addBuild() as well as a garbo job to populate old records.
To post a comment you must log in.
Fine for the ALTER TABLE, but the index creation needs to be done in a separate patch (even if all 4 million+ rows are all NULL, the index build is still to slow for fast downtime).
Best time to build the index is after the column has been populated if you are going to backfill.