On Tue, 2022-03-01 at 08:35 -0500, Stephen Frost wrote:
> * Michael Paquier (michael@paquier.xyz) wrote:
> >
> > Ayway, this function needs to be documented. I think that you should
> > just add that in "Session Information Functions" in func.sgml, same
> > area as current_user(). The last time we talked about the authn ID,
> > one thing we discussed about was how to describe that in a good way to
> > the user, which is why the section of log_connections was reworked a
> > bit. And we don't have yet any references to what an authenticated
> > identity is in the docs.
>
> Agreed that it should be documented and that location seems reasonable
> to me.
Added a first draft in v5, alongside the perltidy fixups mentioned by
Michael.
> > There is no need to update catversion.h in the patch, committers
> > usually take care of that and that's an area of the code that
> > conflicts a lot.
>
> Yeah, best to let committers handle catversion bumps.
Heh, that was added for my benefit -- I was tired of forgetting to
initdb after switching dev branches -- but I've dropped it from the
patch and will just carry that diff locally.
Thanks,
--Jacob
