
Aurélien Bompard has proposed merging lp:~abompard/mailman/pgsql_upgrade into lp:mailman. Requested reviews: Mailman Coders (mailman-coders) For more details, see: https://code.launchpad.net/~abompard/mailman/pgsql_upgrade/+merge/227289 As explained on http://www.mail-archive.com/mailman-developers%40python.org/msg14659.html, the schema upgrade of 20130406 is missing the PostgreSQL part. This branch adds it in a new migration. -- https://code.launchpad.net/~abompard/mailman/pgsql_upgrade/+merge/227289 Your team Mailman Coders is requested to review the proposed merge of lp:~abompard/mailman/pgsql_upgrade into lp:mailman.

Any chance you could write some tests for this branch? I'm basically in favor of it, but tests would be good. OTOH, LP: #1170063 kind of sinks us, doesn't it? Have you looked at Gustavo's comments in that bug? -- https://code.launchpad.net/~abompard/mailman/pgsql_upgrade/+merge/227289 Your team Mailman Coders is requested to review the proposed merge of lp:~abompard/mailman/pgsql_upgrade into lp:mailman.

OK I'll try to write some tests for this migration. There's a workaround for #1170063 in the ticket that works for me, event if it's not perfect. And with the SQLite lockups I reported long ago (#1091586), we can't afford to have neither SQLite nor PostgreSQL support... -- https://code.launchpad.net/~abompard/mailman/pgsql_upgrade/+merge/227289 Your team Mailman Coders is requested to review the proposed merge of lp:~abompard/mailman/pgsql_upgrade into lp:mailman.

Gosh, I'm finally catching up on merge proposals. Aurelien, I think with the switch to SA and Alembic, this one is no longer relevant. -- https://code.launchpad.net/~abompard/mailman/pgsql_upgrade/+merge/227289 Your team Mailman Coders is requested to review the proposed merge of lp:~abompard/mailman/pgsql_upgrade into lp:mailman.

The proposal to merge lp:~abompard/mailman/pgsql_upgrade into lp:mailman has been updated. Status: Needs review => Rejected For more details, see: https://code.launchpad.net/~abompard/mailman/pgsql_upgrade/+merge/227289 -- Your team Mailman Coders is requested to review the proposed merge of lp:~abompard/mailman/pgsql_upgrade into lp:mailman.
participants (2)
-
Aurélien Bompard
-
Barry Warsaw