Re: Allow placeholders in ALTER ROLE w/o superuser - Mailing list pgsql-hackers

From Pavel Borisov
Subject Re: Allow placeholders in ALTER ROLE w/o superuser
Date
Msg-id CALT9ZEH=4gAd7YXaWdGZGrvz8dgrZda_tcRKV389yOd69_JN_Q@mail.gmail.com
Whole thread Raw
In response to Re: Allow placeholders in ALTER ROLE w/o superuser  (Alexander Korotkov <aekorotkov@gmail.com>)
List pgsql-hackers
Hi, Alexander!

On Mon, 5 Dec 2022 at 17:51, Alexander Korotkov <aekorotkov@gmail.com> wrote:
>
> On Mon, Dec 5, 2022 at 2:27 PM Pavel Borisov <pashkin.elfe@gmail.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.

Attachment

pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: move some bitmapset.c macros to bitmapset.h
Next
From: Tom Lane
Date:
Subject: Re: Missing MaterialPath support in reparameterize_path_by_child