Re: psql: Add role's membership options to the \du+ command - Mailing list pgsql-hackers

From David Zhang
Subject Re: psql: Add role's membership options to the \du+ command
Date
Msg-id e79436ce-2321-8ec3-7eab-a8e352ec7085@highgo.ca
Whole thread Raw
In response to Re: psql: Add role's membership options to the \du+ command  ("David G. Johnston" <david.g.johnston@gmail.com>)
List 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.

pgsql-hackers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: [EXTERNAL] Re: [PATCH] Support using "all" for the db user in pg_ident.conf
Next
From: Peter Smith
Date:
Subject: Re: [PATCH] Add pretty-printed XML output option