On Wed, Mar 31, 2021 at 11:18:45AM -0300, Alvaro Herrera wrote: > On 2021-Mar-31, Bruce Momjian wrote: > > > > I assume it is since Alvaro didn't reply. I am planning to apply this > > soon. > > I'm afraid I don't know enough about how parallel query works to make a > good assessment on this being a good approach or not -- and no time at > present to figure it all out.
I'm far from being an expert either, but at the time I wrote it and looking at the code around it probably seemed sensible. We could directly call pgstat_get_my_queryid() in ExecSerializePlan() rather than passing it from the various callers though, at least there would be a single source for it.
