Merge lp:~adeuring/launchpad/fix-broken-initialisation-of-bug-latest-patch-uploaded into lp:launchpad/db-devel
Proposed by
Abel Deuring
Status: | Merged |
---|---|
Merged at revision: | not available |
Proposed branch: | lp:~adeuring/launchpad/fix-broken-initialisation-of-bug-latest-patch-uploaded |
Merge into: | lp:launchpad/db-devel |
Diff against target: |
25 lines (+12/-4) 1 file modified
database/schema/patch-2207-29-0.sql (+12/-4) |
To merge this branch: | bzr merge lp:~adeuring/launchpad/fix-broken-initialisation-of-bug-latest-patch-uploaded |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Stuart Bishop (community) | db | Approve | |
Jonathan Lange (community) | database | Approve | |
Review via email: mp+18610@code.launchpad.net |
To post a comment you must log in.
The original query to initialise the new column bug.latest_ patch_uploaded contained a "teensy weensy little mistake", to quote jtv.
The naively nested SELECTs let the query run for a ridiculously long time. The new version should be better. There is no need to access the tbale Bug at all in the SELECT statement -- all we need are the IDs of bugs that need to be updated.