Mailman 3.0 support for Postgres
Bug #860159 reported by
Stephen A. Goss
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
GNU Mailman |
Fix Released
|
High
|
Barry Warsaw |
Bug Description
Currently, the MM3 source is SQLite specific, but only in the schema and database creation code, otherwise, it uses Storm which (mostly) supports Postgres.
Related branches
lp:~postfuturist/mailman/postgres
- Barry Warsaw: Pending requested
-
Diff: 371 lines (+344/-4)2 files modifiedsrc/mailman/database/mailman_pg.sql (+329/-0)
src/mailman/database/stock.py (+15/-4)
lp:~barry/mailman/postgres-merge
(Merged)
Changed in mailman: | |
status: | New → Confirmed |
importance: | Undecided → High |
assignee: | nobody → Barry Warsaw (barry) |
milestone: | none → 3.0.0b1 |
Changed in mailman: | |
status: | Confirmed → In Progress |
Changed in mailman: | |
status: | In Progress → Fix Committed |
Changed in mailman: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
I've attached a branch with the Postgres support code that I've cooked up.
Some notes:
1. You must install Storm trunk (or v. 0.19 whenever that is released) as version 0.18 is broken w/regards to Postgres and UUID columns.
2. I haven't adapted the test suite to run against Postgres, yet, but my own external integration tests pass (which encompass quite a bit of functionality).
3. There is now an alternate mailman_pg.sql file which is used to create the tables. Currently, two foreign key constraints are commented out because those are violated in Mailman 3 (apparently this doesn't bother SQLite). Some column TYPES are different from mailman.sql. Classes are created in a slightly different order due to FK constraint creation requires the referenced table to actually exist. The primary key indexes defined after each class are probably redundant, as those are created automatically for SERIAL columns defined as PRIMARY KEY in Postgres.
4. Probably more FK constraint violations exist that my tests haven't uncovered.