Date: Tue, 21 Jan 2020 19:37:26 -0300
From: Alvaro Herrera <alvherre@2ndquadrant.com>
To: Alexander Korotkov <a.korotkov@postgrespro.ru>
Cc: Michael Paquier <michael@paquier.xyz>,
	Andres Freund <andres@anarazel.de>,
	Nikita Glukhov <n.gluhov@postgrespro.ru>,
	Thomas Munro <thomas.munro@gmail.com>,
	pgsql-hackers <pgsql-hackers@postgresql.org>,
	Kyotaro HORIGUCHI <horiguchi.kyotaro@lab.ntt.co.jp>,
	David Steele <david@pgmasters.net>
Subject: Re: Psql patch to show access methods info
Message-ID: <20200121223726.GA6714@alvherre.pgsql>
MIME-Version: 1.0
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <20200121223329.GA18317@alvherre.pgsql>
User-Agent: Mutt/1.10.1 (2018-07-13)
Precedence: bulk

On 2020-Jan-21, Alvaro Herrera wrote:

> c) it would be damn handy if \dAf (maybe \dAf+) lists the datatypes that
>    each opfamily has opclasses for.  Maybe make the output an array, like
>    {int4,int8,numeric,...}  Something like [*] but somehow make it
>    prettier?

Sorry, I forgot to copy-edit my text here: I said "make it prettier",
but the query I submitted is already pretty enough ISTM; I had written
that comment when I only had the array_agg() version, but then I changed
it to string_agg() and that seems to have mostly done the trick.  Maybe
improve the format_type() bit to omit the quotes, if possible, but that
doesn't seem a big deal.

-- 
Álvaro Herrera                https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services