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: | Raw Message | Whole Thread | 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)? |