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

Revision history for this message
Stuart Bishop (stub) wrote :


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)

« Back to merge proposal