public inbox for [email protected]  

help / color / mirror / Atom feed

From: Pavel Luzanov <[email protected]>
To: Georgios Kokolatos <[email protected]>
To: [email protected]
Cc: Pavel Luzanov <[email protected]>
Subject: Re: psql: \dl+ to list large objects privileges
Date: Fri, 3 Sep 2021 16:43:43 +0300
Message-ID: <[email protected]> (raw)
In-Reply-To: <163067194414.1167.8140856874411625212.pgcf@coridan.postgresql.org>
References: <[email protected]>
	<[email protected]>
	<163067194414.1167.8140856874411625212.pgcf@coridan.postgresql.org>

Hello,

Thank you very mush for review.

I will prepare a new version of the patch according to your comments. 
For now, I will answer this question:

> I will also inquire as to the need for renaming the function `do_lo_list` to
> `listLargeObjects` and its move to describe.c. from large_obj.c. In itself it is
> not necessarily a blocking point, though it will require some strong arguments
> for doing so.

I understand that I needed a good reason for such actions.

On the one hand all the commands for working with large objects are in 
large_obj.c. On the other hand, all commands for displaying the contents 
of system catalogs are in describe.c. The function do_lo_list belongs to 
both groups.

The main reason for moving the function to describe.c is that I wanted 
to use the printACLColumn function to display lomacl column. 
printACLColumn function is used in all the other commands to display 
privileges and this function is locally defined in describe.c and there 
is no reason to make in public.

Another option is to duplicate the printACLColumn function (or its 
contents) in large_obj.c. This seemed wrong to me.
Is it any other way?

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

view thread (13+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected], [email protected]
  Subject: Re: psql: \dl+ to list large objects privileges
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox