Re: documentation fix for SET ROLE

On Thu, Feb 18, 2021 at 6:18 PM Bossart, Nathan <bossartn(at)amazon(dot)com> wrote:

> On 2/17/21 2:12 PM, David G. Johnston wrote:
> > On Wednesday, February 17, 2021, Bossart, Nathan <bossartn(at)amazon(dot)com
> > <mailto:bossartn(at)amazon(dot)com>> wrote:
> >
> >
> > postgres=# ALTER ROLE test1 SET ROLE test2;
> > ALTER ROLE
> >
> >
> > I would not have expected this to work - “role” isn’t a
> > configuration_parameter. Its actually cool that it does, but this doc
> fix
> > should address this oversight as well.
>
> Here's a patch that adds "role" and "session authorization" as
> configuration parameters, too.
>
>
You will want to add this to the commitfest if you haven't already.

I would suggest adding a section titled "Identification" and placing these
under that.

Reading it over it looks good. One point though: SET and SET ROLE are
indeed "at run-time" (not 'run time'). ALTER ROLE and ALTER DATABASE
should be considered "at connection-time" just like the command-line
options.

David J.