Date: Mon, 27 Sep 2021 21:46:20 -0500
From: Justin Pryzby <pryzby@telsasoft.com>
To: Pavel Stehule <pavel.stehule@gmail.com>
Cc: Laurenz Albe <laurenz.albe@cybertec.at>,
	Ian Lawrence Barwick <barwick@gmail.com>,
	pgsql-hackers@postgresql.org, Gilles Darold <gilles@darold.net>
Subject: Re: [PATCH] psql: \dn+ to show size of each schema (and \dA+ for AMs)
Message-ID: <20210928024620.GO831@telsasoft.com>
References: <20210714030725.GG9600@telsasoft.com>
 <CAB8KJ=gmykiuQKgWpPVQJO69jsSnEKx=1L_+TefEjNWQOL-Oow@mail.gmail.com>
 <2c5c19ffe1a812223b2d60b1248ef31e8b51caaa.camel@cybertec.at>
 <20210716011639.GK20208@telsasoft.com>
 <CAFj8pRDRfiWrnfB9tg5VOeHGjXc11GOw3KVCwyMNZ22jk_nstQ@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: 
 <CAFj8pRDRfiWrnfB9tg5VOeHGjXc11GOw3KVCwyMNZ22jk_nstQ@mail.gmail.com>
User-Agent: Mutt/1.9.4 (2018-02-28)
Archived-At: 
 <https://www.postgresql.org/message-id/20210928024620.GO831%40telsasoft.com>
Precedence: bulk

On Fri, Sep 17, 2021 at 12:05:04PM +0200, Pavel Stehule wrote:
> I can live with the proposed patch, and I understand why  ++ was
> introduced. But I am still not sure it is really user friendly. I prefer to
> extend \dA and \dn with some columns (\dA has only two columns and \dn has
> two columns too), and then we don't need special ++ variants for sizes.
> Using three levels of detail looks not too practical (more when the basic
> reports \dA and \dn) are really very simple).

You're suggesting to include the ACL+description in \dn and handler+description
and \dA.

Another option is to add pg_schema_size() and pg_am_size() without shortcuts in
psql.  That would avoid showing a potentially huge ACL when all one wants is
the schema size, and would serve my purposes well enough to write
| SELECT pg_namespace_size(oid), nspname FROM pg_namespace ORDER BY 1 DESC LIMIT 9;

-- 
Justin