stub | wgrant: new rev of the db patch up, https://pastebin.canonical.com/114208/ contains the before/after schema diff from a dev db. | 11:51 |
---|---|---|
stub | I need a new pastebin bookmark. Lost colorization somewhere. | 11:53 |
wgrant | stub: I don't speak 1970 | 11:54 |
wgrant | But let me see. | 11:54 |
stub | fab | 11:56 |
wgrant | stub: That looks pretty sensible, I think. | 12:21 |
stub | I'll approve and land it now then and we can start the ball rolling next week. Its all standard until we get to the last step. | 12:35 |
stub | Things to confirm on staging are if adding the new column with a FK reference needs to do an unnecessary scan, and similarly that final primary key. | 12:37 |
stub | wgrant: Or do you want to do a proper review first? | 12:38 |
wgrant | stub: I know those two things don't require full scans; I've done them each several times. So we're pretty safe except for the pg_attribute hackery, but that looks fine. | 13:40 |
wgrant | stub: I'll have another look at the patch on Monday before you awake, if you don't mind. Just to ensure it makes sense with a fresh mind. | 13:40 |
wgrant | cprov, cjwatson: Thanks for the reviews. | 13:41 |
cprov | wgrant: sorry for letting them slip until today | 13:42 |
wgrant | cprov: No worries. Could you please also look at https://code.launchpad.net/~wgrant/launchpad/htaccess-in-archiveroot/+merge/225783 and https://code.launchpad.net/~wgrant/launchpad/delete-archiveroot/+merge/225797 soonish? I expect to get moving on migrating production early next week, if possible. | 13:44 |
cprov | wgrant: will do | 13:44 |
cjohnston | wgrant: in Malta we discussed increasing the scan timeout by a minute or two to see if that would help with LP branches... did you have thoughts on the possibility of actually implementing that? | 14:21 |
=== daker_ is now known as daker | ||
=== Ursinha_ is now known as Ursinha | ||
=== cjohnston_ is now known as cjohnston |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!