Re: Allow placeholders in ALTER ROLE w/o superuser

From: Alexander Korotkov <aekorotkov(at)gmail(dot)com>
To: Pavel Borisov <pashkin(dot)elfe(at)gmail(dot)com>
Cc: Steve Chavez <steve(at)supabase(dot)io>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com>, nathandbossart(at)gmail(dot)com, pgsql-hackers(at)postgresql(dot)org
Subject: Re: Allow placeholders in ALTER ROLE w/o superuser
Date: 2022-12-05 13:51:01
Message-ID: CAPpHfducDiwJ2GhWJoh2hqoyeb8mV7B3PdsuK3_HV3H=Mm0eYg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

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.

------
Regards,
Alexander Korotkov

Attachment Content-Type Size
0001-Add-USER-SET-parameter-values-for-pg_db_role_sett-v6.patch application/octet-stream 41.4 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2022-12-05 14:32:59 Re: move some bitmapset.c macros to bitmapset.h
Previous Message Amit Kapila 2022-12-05 13:25:20 Re: [PATCH] Reuse Workers and Replication Slots during Logical Replication