pgsql: Fix brain fade in previous pg_dump patch.

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Fix brain fade in previous pg_dump patch.
Date: 2012-02-10 19:10:14
Message-ID: E1RvvrK-0008BH-Ls@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Fix brain fade in previous pg_dump patch.

In pre-7.3 databases, pg_attribute.attislocal doesn't exist. The easiest
way to make sure the new inheritance logic behaves sanely is to assume it's
TRUE, not FALSE. This will result in printing child columns even when
they're not really needed. We could work harder at trying to reconstruct a
value for attislocal, but there is little evidence that anyone still cares
about dumping from such old versions, so just do the minimum necessary to
have a valid dump.

I had this correct in the original draft of the patch, but for some
unaccountable reason decided it wasn't necessary to change the value.
Testing against an old server shows otherwise...

Branch
------
REL9_0_STABLE

Details
-------
http://git.postgresql.org/pg/commitdiff/54660723715df39ba2c1b71ee0c44012b3d63acc

Modified Files
--------------
src/bin/pg_dump/pg_dump.c | 6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2012-02-10 20:22:55 pgsql: Fix oversight in pg_dump's handling of extension configuration t
Previous Message Tom Lane 2012-02-10 18:28:59 pgsql: Fix pg_dump for better handling of inherited columns.