From: | Jacob Champion <pchampion(at)vmware(dot)com> |
---|---|
To: | "andres(at)anarazel(dot)de" <andres(at)anarazel(dot)de> |
Cc: | "michael(at)paquier(dot)xyz" <michael(at)paquier(dot)xyz>, "rjuju123(at)gmail(dot)com" <rjuju123(at)gmail(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>, "tgl(at)sss(dot)pgh(dot)pa(dot)us" <tgl(at)sss(dot)pgh(dot)pa(dot)us>, "sfrost(at)snowman(dot)net" <sfrost(at)snowman(dot)net> |
Subject: | Re: [PATCH] Expose port->authn_id to extensions and triggers |
Date: | 2022-03-24 17:44:06 |
Message-ID: | 94c53220cb1e8a7a9c411fd14bfc64ff3b65d68d.camel@vmware.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Wed, 2022-03-23 at 16:54 -0700, Andres Freund wrote:
> On 2022-03-23 23:06:14 +0000, Jacob Champion wrote:
> > On Wed, 2022-03-23 at 19:00 -0400, Tom Lane wrote:
> > > Hm. I was more envisioning getting the "sharable" info out of Port
> > > entirely, although I'm not quite sure where it should go instead.
> >
> > If it helps, I can move the substruct out and up to a new global struct
> > (MyProcShared?). At this point I think it's mostly search-and-replace.
>
> Perhaps alongside CurrentUserId etc in miscinit.c? It would be nicer if all
> those were together in a struct, but oh well.
Next draft in v7. My naming choices probably make even less sense now. Any ideas for names for "a bag of stuff that we want parallel workers to have too"?
> Another option would be to make it a GUC. With a bit of care it could be
> automatically synced by the existing parallelism infrastructure...
Like a write-once, PGC_INTERNAL setting? I guess I don't have any
intuition on how that would compare to the separate-global-and-accessor
approach. Is the primary advantage that you don't have to maintain the
serialization logic, or is there more to it?
Thanks,
--Jacob
Attachment | Content-Type | Size |
---|---|---|
since-v6.diff.txt | text/plain | 8.2 KB |
v7-0001-Add-API-to-retrieve-authn_id-from-SQL.patch | text/x-patch | 5.9 KB |
v7-0002-Allow-parallel-workers-to-use-pg_session_authn_id.patch | text/x-patch | 13.6 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Zheng Li | 2022-03-24 17:54:39 | Re: Support logical replication of DDLs |
Previous Message | Tom Lane | 2022-03-24 17:35:20 | Re: Documenting when to retry on serialization failure |