Re: psql: \dl+ to list large objects privileges - Mailing list pgsql-hackers

From Pavel Luzanov
Subject Re: psql: \dl+ to list large objects privileges
Date
Msg-id 47402814-d495-4d9d-dfda-e3d3a67c1ea2@postgrespro.ru
Whole thread Raw
In response to Re: psql: \dl+ to list large objects privileges  (Georgios Kokolatos <gkokolatos@protonmail.com>)
List pgsql-hackers
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




pgsql-hackers by date:

Previous
From: Fujii Masao
Date:
Subject: Re: Improve logging when using Huge Pages
Next
From: Amit Langote
Date:
Subject: Re: Multi-Column List Partitioning