From: | Andres Freund <andres(at)anarazel(dot)de> |
---|---|
To: | Nathan Bossart <nathandbossart(at)gmail(dot)com> |
Cc: | Bertrand Drouvot <bertranddrouvot(dot)pg(at)gmail(dot)com>, Sami Imseih <samimseih(at)gmail(dot)com>, Rahila Syed <rahilasyed90(at)gmail(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Improve LWLock tranche name visibility across backends |
Date: | 2025-08-19 18:06:50 |
Message-ID: | 34drxi6kshtzu5aerftjflrm6wfxxw54emyvy4gqemdpxxuhvm@gymw6its72o7 |
Views: | Whole Thread | Raw Message | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hi,
On 2025-08-19 12:29:14 -0500, Nathan Bossart wrote:
> On Tue, Aug 19, 2025 at 08:09:53AM +0000, Bertrand Drouvot wrote:
> > On Mon, Aug 18, 2025 at 05:53:44PM -0500, Sami Imseih wrote:
> >> > (or some other shmem-based
> >> > data structure we have yet to introduce, like a dslist/dsarray).
> >>
> >> This will be an interesting API to invest time in, if there could be more
> >> use-cases.
> >
> > I did a quick check and I did not find current use cases: possible candidates
> > could be in ExecParallelHashTableAlloc(), PTIterationArray, PTEntryArray for
> > examples but I think they all know the final size upfront so there is no real
> > need for dsarray for those).
> >
> >> I think it's a separate discussion at this point.
> >
> > OTOH, that would be a valid use case to introduce this new API but I'm not sure
> > it's worth it given that the dshash looks good enough for our case (even if not
> > ideal though).
>
> IMHO it'd be okay to proceed with dshash for now. It would be pretty easy
> to switch to something like a "dslist" in the future.
Possibly stupid question - is it really worth having a dynamic structure here?
The number of tranches is strictly bound, it seems like it'd be simpler to
have an array of tranch nmes in shared memory.
Greetings,
Andres Freund
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2025-08-19 18:08:19 | Re: VM corruption on standby |
Previous Message | Álvaro Herrera | 2025-08-19 18:04:18 | Re: New commitfest app release on August 19th |