| From: | Pavel Luzanov <p(dot)luzanov(at)postgrespro(dot)ru> |
|---|---|
| To: | gkokolatos(at)pm(dot)me, Pavel Luzanov <pluzanov(at)postgrespro(dot)ru>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: psql: \dl+ to list large objects privileges |
| Date: | 2021-09-07 11:28:38 |
| Message-ID: | 9729572f-9ae1-23d5-8c6d-e10d6a4973a0@postgrespro.ru |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi,
On 06.09.2021 14:39, gkokolatos(at)pm(dot)me wrote:
> Thanks! The tests look good. A minor nitpick would be to also add a comment on the
> large object to verify that it is picked up correctly.
>
> Also:
>
> +\lo_unlink 42
> +DROP ROLE lo_test;
> +
>
> That last empty line can be removed.
The new version adds a comment to a large object and removes the last empty line.
--
Pavel Luzanov
Postgres Professional: https://postgrespro.com
The Russian Postgres Company
| Attachment | Content-Type | Size |
|---|---|---|
| lo-list-acl-v3.patch | text/x-patch | 10.8 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | kuroda.hayato@fujitsu.com | 2021-09-07 11:30:27 | RE: Allow escape in application_name (was: [postgres_fdw] add local pid to fallback_application_name) |
| Previous Message | Bharath Rupireddy | 2021-09-07 11:22:27 | Re: What are exactly bootstrap processes, auxiliary processes, standalone backends, normal backends(user sessions)? |