Skip to content

Commit 719304a

Browse files
tglsfdctglsfdc
committed
Fix minor issues in psql's new \dAc and related commands.
The type-name pattern in \dAc and \dAf was matched only to the actual pg_type.typname string, which is fairly user-unfriendly in cases where that is not what's shown to the user by format_type (compare "_int4" and "integer[]"). Make this code match what \dT does, i.e. match the pattern against either typname or format_type() output. Also fix its broken handling of schema-name restrictions. (IOW, make these processSQLNamePattern calls match \dT's.) While here, adjust whitespace to make the query a little prettier in -E output, too. Also improve some inaccuracies and shaky grammar in the related documentation. Noted while working on a patch for intarray's opclasses; I wondered why I couldn't get a match to "integer*" for the input type name.
1 parent 22c1055 commit 719304a

File tree

3 files changed

+48
-27
lines changed

3 files changed

+48
-27
lines changed

‎doc/src/sgml/indices.sgml

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1410,6 +1410,15 @@ SELECT am.amname AS index_method,
14101410
ORDER BY index_method, opfamily_name, opfamily_operator;
14111411
</programlisting>
14121412
</para>
1413+
1414+
<tip>
1415+
<para>
1416+
<xref linkend="app-psql"/> has
1417+
commands <command>\dAc</command>, <command>\dAf</command>,
1418+
and <command>\dAo</command>, which provide slightly more sophisticated
1419+
versions of these queries.
1420+
</para>
1421+
</tip>
14131422
</sect1>
14141423

14151424

‎doc/src/sgml/ref/psql-ref.sgml

Lines changed: 20 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -1245,13 +1245,13 @@ testdb=&gt;
12451245
<listitem>
12461246
<para>
12471247
Lists operator classes
1248-
(see <xref linkend="catalog-pg-opclass"/>).
1248+
(see <xref linkend="xindex-opclass"/>).
12491249
If <replaceable class="parameter">access-method-pattern</replaceable>
12501250
is specified, only operator classes associated with access methods whose
1251-
names match the pattern are listed.
1251+
names match that pattern are listed.
12521252
If <replaceable class="parameter">input-type-pattern</replaceable>
12531253
is specified, only operator classes associated with input types whose
1254-
names match the pattern are listed.
1254+
names match that pattern are listed.
12551255
If <literal>+</literal> is appended to the command name, each operator
12561256
class is listed with its associated operator family and owner.
12571257
</para>
@@ -1268,13 +1268,13 @@ testdb=&gt;
12681268
<listitem>
12691269
<para>
12701270
Lists operator families
1271-
(see <xref linkend="catalog-pg-opfamily"/>).
1271+
(see <xref linkend="xindex-opfamily"/>).
12721272
If <replaceable class="parameter">access-method-pattern</replaceable>
12731273
is specified, only operator families associated with access methods whose
1274-
names match the pattern are listed.
1274+
names match that pattern are listed.
12751275
If <replaceable class="parameter">input-type-pattern</replaceable>
12761276
is specified, only operator families associated with input types whose
1277-
names match the pattern are listed.
1277+
names match that pattern are listed.
12781278
If <literal>+</literal> is appended to the command name, each operator
12791279
family is listed with its owner.
12801280
</para>
@@ -1292,15 +1292,15 @@ testdb=&gt;
12921292
<listitem>
12931293
<para>
12941294
Lists operators associated with operator families
1295-
(<xref linkend="catalog-pg-amop"/>).
1295+
(see <xref linkend="xindex-strategies"/>).
12961296
If <replaceable class="parameter">access-method-pattern</replaceable>
12971297
is specified, only members of operator families associated with access
1298-
methods whose names match the pattern are listed.
1299-
If <replaceable class="parameter">input-type-pattern</replaceable>
1300-
is specified, only members of operator families whose names match the
1298+
methods whose names match that pattern are listed.
1299+
If <replaceable class="parameter">operator-family-pattern</replaceable>
1300+
is specified, only members of operator families whose names match that
13011301
pattern are listed.
13021302
If <literal>+</literal> is appended to the command name, each operator
1303-
is listed with its strategy number, purpose and sort operator family.
1303+
is listed with its sort operator family (if it is an ordering operator).
13041304
</para>
13051305
</listitem>
13061306
</varlistentry>
@@ -1314,14 +1314,16 @@ testdb=&gt;
13141314
</term>
13151315
<listitem>
13161316
<para>
1317-
Lists functions associated with operator families
1318-
(<xref linkend="catalog-pg-amproc"/>).
1317+
Lists support functions associated with operator families
1318+
(see <xref linkend="xindex-support"/>).
13191319
If <replaceable class="parameter">access-method-pattern</replaceable>
1320-
is specified, only members of operator families associated with access
1321-
methods whose names match the pattern are listed.
1322-
If <replaceable class="parameter">input-type-pattern</replaceable>
1323-
is specified, only members of operator families whose names match the
1324-
pattern are listed.
1320+
is specified, only functions of operator families associated with
1321+
access methods whose names match that pattern are listed.
1322+
If <replaceable class="parameter">operator-family-pattern</replaceable>
1323+
is specified, only functions of operator families whose names match
1324+
that pattern are listed.
1325+
If <literal>+</literal> is appended to the command name, functions are
1326+
displayed verbosely, with their actual parameter lists.
13251327
</para>
13261328
</listitem>
13271329
</varlistentry>

‎src/bin/psql/describe.c

Lines changed: 19 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -6050,7 +6050,7 @@ printACLColumn(PQExpBuffer buf, const char *colname)
60506050
* \dAc
60516051
* Lists operator classes
60526052
*
6053-
* Takes an optional regexps to filter by index access method and type.
6053+
* Takes optional regexps to filter by index access method and input data type.
60546054
*/
60556055
bool
60566056
listOperatorClasses(const char *access_method_pattern,
@@ -6104,6 +6104,7 @@ listOperatorClasses(const char *access_method_pattern,
61046104
" LEFT JOIN pg_catalog.pg_am am on am.oid = c.opcmethod\n"
61056105
" LEFT JOIN pg_catalog.pg_namespace n ON n.oid = c.opcnamespace\n"
61066106
" LEFT JOIN pg_catalog.pg_type t ON t.oid = c.opcintype\n"
6107+
" LEFT JOIN pg_catalog.pg_namespace tn ON tn.oid = t.typnamespace\n"
61076108
);
61086109
if (verbose)
61096110
appendPQExpBuffer(&buf,
@@ -6114,8 +6115,13 @@ listOperatorClasses(const char *access_method_pattern,
61146115
have_where = processSQLNamePattern(pset.db, &buf, access_method_pattern,
61156116
false, false, NULL, "am.amname", NULL, NULL);
61166117
if (type_pattern)
6118+
{
6119+
/* Match type name pattern against either internal or external name */
61176120
processSQLNamePattern(pset.db, &buf, type_pattern, have_where, false,
6118-
NULL, "t.typname", NULL, NULL);
6121+
"tn.nspname", "t.typname",
6122+
"pg_catalog.format_type(t.oid, NULL)",
6123+
"pg_catalog.pg_type_is_visible(t.oid)");
6124+
}
61196125

61206126
appendPQExpBufferStr(&buf, "ORDER BY 1, 2, 4;");
61216127
res = PSQLexec(buf.data);
@@ -6139,7 +6145,7 @@ listOperatorClasses(const char *access_method_pattern,
61396145
* \dAf
61406146
* Lists operator families
61416147
*
6142-
* Takes an optional regexps to filter by index access method and type.
6148+
* Takes optional regexps to filter by index access method and input data type.
61436149
*/
61446150
bool
61456151
listOperatorFamilies(const char *access_method_pattern,
@@ -6184,15 +6190,19 @@ listOperatorFamilies(const char *access_method_pattern,
61846190
if (type_pattern)
61856191
{
61866192
appendPQExpBuffer(&buf,
6187-
"\n %s EXISTS (\n"
6193+
" %s EXISTS (\n"
61886194
" SELECT 1\n"
61896195
" FROM pg_catalog.pg_type t\n"
61906196
" JOIN pg_catalog.pg_opclass oc ON oc.opcintype = t.oid\n"
6191-
" WHERE oc.opcfamily = f.oid",
6197+
" LEFT JOIN pg_catalog.pg_namespace tn ON tn.oid = t.typnamespace\n"
6198+
" WHERE oc.opcfamily = f.oid\n",
61926199
have_where ? "AND" : "WHERE");
6200+
/* Match type name pattern against either internal or external name */
61936201
processSQLNamePattern(pset.db, &buf, type_pattern, true, false,
6194-
NULL, "t.typname", NULL, NULL);
6195-
appendPQExpBuffer(&buf, ")");
6202+
"tn.nspname", "t.typname",
6203+
"pg_catalog.format_type(t.oid, NULL)",
6204+
"pg_catalog.pg_type_is_visible(t.oid)");
6205+
appendPQExpBuffer(&buf, " )\n");
61966206
}
61976207

61986208
appendPQExpBufferStr(&buf, "ORDER BY 1, 2;");
@@ -6217,7 +6227,7 @@ listOperatorFamilies(const char *access_method_pattern,
62176227
* \dAo
62186228
* Lists operators of operator families
62196229
*
6220-
* Takes an optional regexps to filter by index access method and operator
6230+
* Takes optional regexps to filter by index access method and operator
62216231
* family.
62226232
*/
62236233
bool
@@ -6304,7 +6314,7 @@ listOpFamilyOperators(const char *access_method_pattern,
63046314
* \dAp
63056315
* Lists support functions of operator families
63066316
*
6307-
* Takes an optional regexps to filter by index access method and operator
6317+
* Takes optional regexps to filter by index access method and operator
63086318
* family.
63096319
*/
63106320
bool

0 commit comments

Comments
 (0)
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy