Subject: Re: psql: \dl+ to list large objects privileges
To: Georgios Kokolatos <gkokolatos@protonmail.com>,
 pgsql-hackers@lists.postgresql.org
Cc: Pavel Luzanov <pluzanov@postgrespro.ru>
References: <6d722115-6297-bc53-bb7f-5f150e765299@postgrespro.ru>
 <163067194414.1167.8140856874411625212.pgcf@coridan.postgresql.org>
 <163067311464.1167.13782276673681988632.pgcf@coridan.postgresql.org>
From: Pavel Luzanov <p.luzanov@postgrespro.ru>
Message-ID: <943dfff6-bcda-4481-0cbf-0dd26b8949a8@postgrespro.ru>
Date: Fri, 3 Sep 2021 16:20:09 +0300
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101
 Thunderbird/78.11.0
MIME-Version: 1.0
In-Reply-To: 
 <163067311464.1167.13782276673681988632.pgcf@coridan.postgresql.org>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 7bit
Content-Language: en-US
Archived-At: 
 <https://www.postgresql.org/message-id/943dfff6-bcda-4481-0cbf-0dd26b8949a8%40postgrespro.ru>
Precedence: bulk

Hello,

Thank you very much for review.

> Since '\dl' already contains description, it might be worthwhile to consider to add the column `Access privileges`
> without introducing a verbose version.

I thought about it.
The reason why I decided to add the verbose version is because of 
backward compatibility. Perhaps the appearance of a new column in an 
existing command may become undesirable to someone.

If we don't worry about backward compatibility, the patch will be 
easier. But I'm not sure this is the right approach.

Pavel Luzanov
Postgres Professional: https://postgrespro.com
The Russian Postgres Company