On Thu, Apr 01, 2021 at 11:05:24PM +0800, Julien Rouhaud wrote:
> 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.
Here's a v21 that includes the mentioned change.