Date: Fri, 13 Sep 2019 18:36:12 -0300
From: Alvaro Herrera <alvherre@2ndquadrant.com>
To: Alexander Korotkov <a.korotkov@postgrespro.ru>
Cc: 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>,
	Michael Paquier <michael@paquier.xyz>
Subject: Re: Psql patch to show access methods info
Message-ID: <20190913213612.GA20762@alvherre.pgsql>
MIME-Version: 1.0
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: 
 <CAPpHfdsvVxC+MF-JQp-GFq4xQQ-DhUDrLizK+OSAhr_VFpsp1w@mail.gmail.com>
User-Agent: Mutt/1.9.4 (2018-02-28)
Precedence: bulk

On 2019-Aug-06, Alexander Korotkov wrote:

> Revised patch is attached.  Changes to \dA+ command are reverted.  It
> also contains some minor improvements.
> 
> Second patch looks problematic for me, because it provides index
> description alternative to \d+.  IMHO, if there is something really
> useful to display about index, we should keep it in \d+.  So, I
> propose to postpone this.

Are you saying that we should mark this entire CF entry as Returned with
Feedback?  Or do you see a subset of your latest 0001 as a commitable
patch?

Thanks

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