]> granicus.if.org Git - postgresql/commit
Fix brain fade in previous pg_dump patch.
authorTom Lane <tgl@sss.pgh.pa.us>
Fri, 10 Feb 2012 19:09:37 +0000 (14:09 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Fri, 10 Feb 2012 19:09:37 +0000 (14:09 -0500)
commite1fcf16b72faea2ba28ddc31441b9b0eb2ba5970
tree904d86e45d670dcfd62d4fbff5f2215d54f1a495
parent0951f4d1f5849ff3d868452debf4ced1a736efd0
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...
src/bin/pg_dump/pg_dump.c