Horton

Re: pg_dump: fail to restore partition table with serial type

Alvaro Herrera <alvherre@2ndquadrant.com>

View thread View on postgr.es
From: Alvaro Herrera <alvherre@2ndquadrant.com>
To: Rushabh Lathia <rushabh.lathia@gmail.com>
Cc: PostgreSQL Hackers <pgsql-hackers@postgresql.org>
Date: 2019-06-10T23:07:52Z
Lists: pgsql-hackers 
On 2019-Jun-07, Alvaro Herrera wrote:

> I looked for other cases that could have been broken by changing the
> partition creation methodology in pg_dump, and didn't find anything.
> That part of pg_dump (dumpTableSchema) is pretty spaghettish, though;
> the fact that shouldPrintColumn makes some partitioned-related decisions
> and then dumpTableSchema make them again is notoriously confusing.  I
> could have easily missed something.

There was indeed one more problem, that only the pg10 pg_upgrade test
detected.  Namely, binary-upgrade dump didn't restore for locally
defined constraints: they were dumped twice, first in the table
definition and later by the ALTER TABLE ADD CONSTRAINT bit for binary
upgrade that I had failed to notice.  Ooops.  The reason pg10 detected
it and the other branches didn't, is that the only constraint of this
ilk that remained after running regress was removed by 05bd889904e0 :-(

Pushed to the three branches.  Hopefully it won't explode as
spectacularly this time ...

-- 
Álvaro Herrera                https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

Commits

  1. Make pg_dump emit ATTACH PARTITION instead of PARTITION OF (reprise)

  2. Revert "Make pg_dump emit ATTACH PARTITION instead of PARTITION OF"

  3. Make pg_dump emit ATTACH PARTITION instead of PARTITION OF

  4. Allow a partitioned table to have a default partition.

  5. Give partitioned table "p" in regression tests a less generic name.