Re: pg_dump is broken for partition tablespaces
Alvaro Herrera <alvherre@2ndquadrant.com>
From: Alvaro Herrera <alvherre@2ndquadrant.com>
To: Andres Freund <andres@anarazel.de>
Cc: Tom Lane <tgl@sss.pgh.pa.us>, David Rowley <david.rowley@2ndquadrant.com>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2019-04-14T17:32:14Z
Lists: pgsql-hackers
On 2019-Apr-14, Andres Freund wrote: > On 2019-04-14 10:38:05 -0400, Tom Lane wrote: > > It's entirely possible BTW that this whole business of inheriting > > tablespace from the partitioned table is broken and should be thrown > > out. I certainly don't see any compelling reason for partitions to > > act differently from regular tables in this respect, and the more > > problems we find with the idea, the less attractive it seems. > > Indeed. After discovering during the tableam work, and trying to write > tests for the equivalent feature for tableam, I decided that just not > allowing AM specifications for partitioned tables is the right call - at > least until the desired behaviour is clearer. The discussion of the last > few days makes me think so even more. To be honest, when doing that feature I neglected to pay attention to (read: forgot about) default_tablespace, and it's mostly the interactions with that feature that makes this partitioned table stuff so complicated. I'm not 100% convinced yet that we need to throw it out completely, but I'm less sure now about it than I was before. -- Álvaro Herrera https://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
Commits
-
Make pg_dump emit ATTACH PARTITION instead of PARTITION OF (reprise)
- 6a781c4f5fec 11.4 landed
- 33a53130a894 12.0 landed
- 1eb8a5ea463d 10.9 landed
-
Fix tablespace inheritance for partitioned rels
- 87259588d0ab 12.0 landed
-
Make pg_dump emit ATTACH PARTITION instead of PARTITION OF
- a98c48debcd0 11.3 landed
- 5a191f697400 10.8 landed
- 3b23552ad8bb 12.0 landed
-
Make sure ALTER TABLE preserves index tablespaces.
- bd673e8e864a 10.0 cited