Code review comment for lp:~deryck/launchpad/product-specification-sharing-policy-idx-1057617

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

The DB patch is fine, but we can't think why it is needed. This index is really only helpful for selecting all products with a particular policy in an arbitrary order, which might be useful for initial population of the column but probably not given the Product table doesn't have that many rows.

review: Needs Information (db)

« Back to merge proposal