Launchpad itself

Code review comment for lp:~gary/launchpad/bug164196-3

Revision history for this message

Stuart Bishop (stub) wrote on 2011-02-17:

patch-2208-45-0.sql

Please add the following statement after the ALTER TABLE (all rows are going to be rewritten, so we want to repack the table):

CLUSTER BugNotification USING bugnotification__date_emailed__idx

Bug notification is regularly trimmed, so there are not too many rows to deal with - should be fast enough for a db patch.

I prefer status rather than the boolean too.

review: Approve (db)