]> granicus.if.org Git - postgresql/commit
Fix old pg_dump oversight: default values for domains really need to be dumped
authorTom Lane <tgl@sss.pgh.pa.us>
Tue, 21 Feb 2006 18:01:41 +0000 (18:01 +0000)
committerTom Lane <tgl@sss.pgh.pa.us>
Tue, 21 Feb 2006 18:01:41 +0000 (18:01 +0000)
commit6f96c539ede267ab31802da264e5c2bc226784a5
tree92f1f94fafb49937b6b292b859e78f1fd4a2b8cd
parenta386cd7806a90d5f45e5801f1d0ce717f529ffd2
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.
src/bin/pg_dump/pg_dump.c