Re: Psql patch to show access methods info - Mailing list pgsql-hackers

From Arthur Zakirov
Subject Re: Psql patch to show access methods info
Date
Msg-id c94f871f-12e5-304e-a6b1-08eab7a94187@postgrespro.ru
Whole thread Raw
In response to Re: Psql patch to show access methods info  (s.cherkashin@postgrespro.ru)
List pgsql-hackers
Hello,

On 20.11.2018 16:08, s.cherkashin@postgrespro.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


pgsql-hackers by date:

Previous
From: Tomas Vondra
Date:
Subject: Re: mysql_fdw crash
Next
From: John Naylor
Date:
Subject: Re: [RFC] Removing "magic" oids