From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Mark Dilger <mark(dot)dilger(at)enterprisedb(dot)com> |
Cc: | Andrew Dunstan <andrew(at)dunslane(dot)net>, Peter Eisentraut <peter(dot)eisentraut(at)enterprisedb(dot)com>, Joshua Brindle <joshua(dot)brindle(at)crunchydata(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, Jeff Davis <pgsql(at)j-davis(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Joe Conway <joe(at)crunchydata(dot)com> |
Subject: | Re: Granting SET and ALTER SYSTE privileges for GUCs |
Date: | 2022-03-28 19:11:15 |
Message-ID: | 895087.1648494675@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Mark Dilger <mark(dot)dilger(at)enterprisedb(dot)com> writes:
> Version 12 also introduces \dcp (pneumonic, "Describe Configuration Parameter") for listing parameters, with \dcp+ also showing the acl, like:
The fact that that code is not dry behind the ears is painfully obvious.
It's not documented in psql-ref, not tested anywhere AFAICS, and its
handling of the pattern parameter is inconsistent with every other
\d command. The wildcard character should be * not %. It only
accidentally fails to dump core if no pattern is given, too.
> \dcp[+] only shows "user" and "superuser" parameters:
Why make that restriction? Also, I find it astonishing that this doesn't
show the GUC's value by default. The non-plus form of the command seems
useless as it stands, or at least its intended use-case is so narrow
I can't see it. If we're to have it at all, it seems like it ought to
be a reasonably useful shortcut for interrogating pg_settings. I'd
expect the base set of columns to be name, value, and possibly unit;
then add ACL with +. I'm not sure that GucContext belongs in this at all,
but if it does, it's a + column.
On the whole perhaps this should be taken out again; it's a bit
late in the cycle to be introducing new features, especially ones
as subject to bikeshedding as a \d command is. My ideas about what
columns to show probably wouldn't match anyone else's ... and we
haven't even gotten to whether \dcp is an okay choice of name.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2022-03-28 19:14:04 | Re: Granting SET and ALTER SYSTE privileges for GUCs |
Previous Message | Greg Stark | 2022-03-28 19:09:12 | Re: [PATCH] Add extra statistics to explain for Nested Loop |