]> granicus.if.org Git - postgresql/commit
Fix underqualified cast-target type names in pg_dump and psql queries.
authorTom Lane <tgl@sss.pgh.pa.us>
Tue, 31 Oct 2017 17:40:23 +0000 (13:40 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Tue, 31 Oct 2017 17:40:23 +0000 (13:40 -0400)
commit9cf2b854a59d227eaec3d97780e986ebdd0c6854
treef888cc28878a1b60f152d8264435a3d5ce4d3aca
parent7becb5fa1d8760ee70258ff23ce229ce5451c597
Fix underqualified cast-target type names in pg_dump and psql queries.

Queries running with some non-pg_catalog schema frontmost in their search
path need to be careful to schema-qualify type names that should be sought
in pg_catalog.  Vitaly Burovoy reported an oversight of this sort in
pg_dump's dumpSequence, and grepping detected another one in psql's
describeOneTableDetails, both introduced by sequence-related changes in
v10.  In pg_dump, we can fix things by removing the cast altogether, since
it doesn't really matter what data types are reported for these query
result columns.  Likewise in psql, the query seemed to be working unduly
hard to get a result that's guaranteed to be exactly 'bigint'.

I also changed a couple of occurrences of "::char" similarly.  These are
not bugs, since "char" is a typename keyword and not subject to search_path
rules, but it seems better to use uniform style.

Vitaly Burovoy and Tom Lane

Discussion: https://postgr.es/m/CAKOSWN=ds66zLw2SqkLTM8wbXFgDbc_OdkmT3dJfPT2mE5kipA@mail.gmail.com
src/bin/pg_dump/pg_dump.c
src/bin/psql/describe.c