On 2025-Jul-16, Michael Paquier wrote: > On Wed, Jul 16, 2025 at 09:30:31AM +0000, Bertrand Drouvot wrote:
> > Indeed, the wait event names are MultixactOffsetSLRU and > > MultixactMemberSLRU in > > the C file but MultiXactOffsetSLRU and MultiXactMemberSLRU in the text file. > > > > That breaks joins between pg_stat_activity and pg_wait_events on the wait > > event name. PFA a patch to fix those (I'm not able to find other discrepancy > > with the "automated" work I'm doing in [1], so I'm confident there is no > > others). > > This is in the same line as 08b9b9e043bb, down to v17. For > 08b9b9e043bb, we were lucky enough to not be in GA, would people mind > to sneak this lwlock tranche name change in v17? I don't. These names are unlikely to be anywhere other than in the output of queries for any length of time, so if we change them now, nothing will break permanently. I grant that there might be small temporary breakage if somebody is storing wait event samples or similar, but I doubt it'll be a problem to change it. Long-term lack of joinability between pg_stat_activity and pg_wait_events in the 17 branch would likely be a bigger problem. -- Álvaro Herrera Breisgau, Deutschland — https://www.EnterpriseDB.com/ "The problem with the facetime model is not just that it's demoralizing, but that the people pretending to work interrupt the ones actually working." -- Paul Graham, http://www.paulgraham.com/opensource.html