Re: psql: Add role's membership options to the \du+ command
David Zhang <david.zhang@highgo.ca>
From: David Zhang <david.zhang@highgo.ca>
To: "David G. Johnston" <david.g.johnston@gmail.com>,
Pavel Luzanov <p.luzanov@postgrespro.ru>
Cc: "pgsql-hackers@lists.postgresql.org" <pgsql-hackers@lists.postgresql.org>
Date: 2023-02-15T22:52:20Z
Lists: pgsql-hackers
On 2023-02-15 1:37 p.m., David G. Johnston wrote: > On Wed, Feb 15, 2023 at 2:31 PM David Zhang <david.zhang@highgo.ca> wrote: > > There is a default built-in role `pg_monitor` and the behavior > changed after the patch. If `\dg+` and `\du+` is treated as the > same, and `make check` all pass, then I assume there is no test > case to verify the output of `duS+`. My point is should we > consider add a test case? > > I mean, either you accept the change in how this meta-command presents > its information or you don't. I don't see how a test case is > particularly beneficial. Or, at least the pg_monitor role is not > special in this regard. Alice changed too and you don't seem to be > including it in your complaint. Good improvement, +1.
Commits
-
Add psql \drg command to display role grants.
- d65ddaca93f6 17.0 landed
- 0a1d2a7df852 16.0 landed