Re: Psql patch to show access methods info

From: Arthur Zakirov <a(dot)zakirov(at)postgrespro(dot)ru>
To: s(dot)cherkashin(at)postgrespro(dot)ru, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>
Cc: Michael Paquier <michael(at)paquier(dot)xyz>, Nikita Glukhov <n(dot)gluhov(at)postgrespro(dot)ru>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: Psql patch to show access methods info
Date: 2018-11-20 14:32:57
Message-ID: c94f871f-12e5-304e-a6b1-08eab7a94187@postgrespro.ru
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hello,

On 20.11.2018 16:08, s(dot)cherkashin(at)postgrespro(dot)ru wrote:
> Ok, I fixed this.

I looked at the patch. It is in good shape. It compiles and tests are
passed.

I have few a questions related with throwing errors. They might be silly :)

\dAp as well as \dA command throw an error if a server's version below 9.6:

"The server (version %s) does not support access methods"

But other \dA commands don't. It seems that there is enough information
in catalog for servers below 9.6. That is there are pg_am, pg_opfamily,
pg_amop and other catalog tables related with access methods.

\dAp calls pg_indexam_has_property() function, which doesn't exist in
servers 9.5 and below. Is this the reason that it throws an error? If so
then describeOneIndexColumnProperties() also should throw an error,
because it calls pg_index_column_has_property() function, which doesn't
exist in servers 9.5 and below.

What do you think?

--
Arthur Zakirov
Postgres Professional: http://www.postgrespro.com
Russian Postgres Company

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message John Naylor 2018-11-20 14:49:27 Re: [RFC] Removing "magic" oids
Previous Message Tomas Vondra 2018-11-20 14:29:35 Re: mysql_fdw crash