Code review comment for lp:~stevenk/launchpad/db-searchablenames-for-pu

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

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.

review: Approve (db)

« Back to merge proposal