Code review comment for lp:~michael.nelson/launchpad/db-changes-build-generalisation-new

Revision history for this message
Michael Nelson (michael.nelson) wrote :

Thanks Bjorn. I've another question for you both: the patch moves the old build table to todrop.build and deletes its constraints.

Should I have a contingency patch that (1) restores the old table, (2) restores its constraints (3) moves the new tables to the todrop namespace?

I'm kindof nervous about landing this as I don't see a way to back it out if need be (given the size etc.)

I'll get this tested today on dogfood.

« Back to merge proposal