diff options
author | Tom Lane <tgl@sss.pgh.pa.us> | 2017-07-26 19:35:35 -0400 |
---|---|---|
committer | Tom Lane <tgl@sss.pgh.pa.us> | 2017-07-26 19:36:19 -0400 |
commit | a2fc3431c35504c022d9e1671d228f734eddcbcf (patch) | |
tree | 8bfba7cbf8ed520d3fbaeba37eaecd2ff9e2191f /src/backend/parser/parse_utilcmd.c | |
parent | 8c348765f95e97a9a7774467c744a33da21206fd (diff) | |
download | postgresql-a2fc3431c35504c022d9e1671d228f734eddcbcf.tar.gz postgresql-a2fc3431c35504c022d9e1671d228f734eddcbcf.zip |
Clean up SQL emitted by psql/describe.c.
Fix assorted places that had not bothered with the convention of
prefixing catalog and function names with "pg_catalog.". That
could possibly result in query failure when running with a nondefault
search_path. Also fix two places that weren't quoting OID literals.
I think the latter hasn't mattered much since about 7.3, but it's still
a bad idea to be doing it in 99 places and not in 2 others.
Also remove a useless EXISTS sub-select that someone had stuck into
describeOneTableDetails' queries for child tables. We just got the OID
out of pg_class, so I hardly see how checking that it exists in pg_class
was doing anything helpful.
In passing, try to improve the emitted formatting of a couple of
these queries, though I didn't work really hard on that. And merge
unnecessarily duplicative coding in some other places.
Much of this was new in HEAD, but some was quite old; back-patch
as appropriate.
Diffstat (limited to 'src/backend/parser/parse_utilcmd.c')
0 files changed, 0 insertions, 0 deletions