public inbox for [email protected]  

help / color / mirror / Atom feed

From: Tom Lane <[email protected]>
To: Laurenz Albe <[email protected]>
Cc: David G. Johnston <[email protected]>
Cc: Erik Wienhold <[email protected]>
Cc: PostgreSQL Hackers <[email protected]>
Subject: Re: Fix output of zero privileges in psql
Date: Tue, 17 Oct 2023 12:33:09 -0400
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>
	<[email protected]>
	<4tgcecleqgkynyat7t3innrsdpopeiukumb7s4f2gdw3zuzidb@27urnfsqebdy>
	<[email protected]>
	<hltc5k6wgxkrrsd5wuzt45elxr6wiyvgqkixmfqfkoxirpplsu@zt4uhz5yhdy4>
	<[email protected]>
	<lir6jx6hjhsqkskekrhtoflbjbedv7wqowvz5wc2a742by5i55@uqeiwszaksvs>
	<[email protected]>
	<5uytof65cguide4mjhxzakd5w74hpuppjsrqfwuywtcf2r2xwl@lbi4fc374nig>
	<[email protected]>
	<cww6udzxvxorpznocvufq42mly3qyi3wcjx45nhlbj3wm6rjfc@36pzmsfodow5>
	<[email protected]>
	<CAKFQuwbWt6pvSFERENDh-v8bsahgSd1NDfxsRfFT4g_MNDu+ZQ@mail.gmail.com>
	<[email protected]>

Laurenz Albe <[email protected]> writes:
> On Mon, 2023-10-16 at 19:05 -0700, David G. Johnston wrote:
>> Reading both threads I'm not seeing any specific rejection of the
>> solution that we simply represent empty privileges as "(none)".

> Thanks for that summary.  I prefer my version (simply display NULLs
> as NULLs), but I am not wedded to it.  I had the impression that Tom
> would prefer that too, but is woried if the incompatibility introduced
> would outweigh the small benefit (of either patch).

> So it is clear that we don't have a consensus.

FWIW, my druthers are to make the describe.c queries honor \pset null
(not only for privileges, but anywhere else they fail to) and do
nothing beyond that.  I think that'll generally reduce the surprise
factor, while anything else we might opt to do will increase it.

If that fails to garner a consensus, my second choice would be to
do that plus translate empty-but-not-null ACLs to "(none)".

			regards, tom lane

view thread (2+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected], [email protected], [email protected]
  Subject: Re: Fix output of zero privileges in psql
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox