Re: Allow placeholders in ALTER ROLE w/o superuser

Hi, Alexander!

On Mon, 5 Dec 2022 at 17:51, Alexander Korotkov <aekorotkov(at)gmail(dot)com> wrote:
>
> On Mon, Dec 5, 2022 at 2:27 PM Pavel Borisov <pashkin(dot)elfe(at)gmail(dot)com> wrote:
> > After posting the patch I've found my own typo in docs. So corrected
> > it in v5 (PFA).
>
> The new revision of the patch is attached.
>
> I've removed the mention of "(s)" suffix from the "Server
> Configuration" docs section. I think it might be confusing since this
> suffix isn't a part of the variable name. It is only used for storage.
> Instead, I've added the description of this suffix to the catalog
> structure description and psql documentation.
>
> Also, I've added psql tab completion for the USER SET flag, and made
> some enhancements to comments, tests, and commit message.

The changes in expected test results are somehow lost in v6, I've
corrected them in v7.
Otherwise, I've looked through the updated patch and it is good.

Regards,
Pavel.