pgsql: Fix old pg_dump oversight: default values for domains really need - Mailing list pgsql-committers

From tgl@postgresql.org (Tom Lane)
Subject pgsql: Fix old pg_dump oversight: default values for domains really need
Date
Msg-id 20060221180132.B55BD9DCA1C@postgresql.org
Whole thread Raw
List pgsql-committers
Log Message:
-----------
Fix old pg_dump oversight: default values for domains really need to be dumped
by decompiling the typdefaultbin expression, not just printing the typdefault
text which may be out-of-date or assume the wrong schema search path.  (It's
the same hazard as for adbin vs adsrc in column defaults.)  The catalogs.sgml
spec for pg_type implies that the correct procedure is to look to
typdefaultbin first and consider typdefault only if typdefaultbin is NULL.
I made dumping of both domains and base types do that, even though in the
current backend code typdefaultbin is always correct for domains and
typdefault for base types --- might as well try to future-proof it a little.
Per bug report from Alexander Galler.

Modified Files:
--------------
    pgsql/src/bin/pg_dump:
        pg_dump.c (r1.429 -> r1.430)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/bin/pg_dump/pg_dump.c.diff?r1=1.429&r2=1.430)

pgsql-committers by date:

Previous
From: tgl@postgresql.org (Tom Lane)
Date:
Subject: pgsql: Adjust probe for getaddrinfo to cope with macro-ized definitions,
Next
From: tgl@postgresql.org (Tom Lane)
Date:
Subject: pgsql: Fix old pg_dump oversight: default values for domains really need