James Coleman <jtc331@gmail.com> writes: > I'm looking at this now, and realized that at least for parallel plans the > current patch tracks the tuplesort instrumentation whether or not an > EXPLAIN ANALYZE is in process.
> Is this fairly standard for executor nodes? Or is it expected to condition > some of this tracking based on whether or not an ANALYZE is running?
No, it's entirely not standard. Maybe you could make an argument that it's too cheap to bother making it conditional, but without a convincing argument for that, it needs to be conditional.
That's what I figured, but as I mentioned I've having trouble figuring out how the fact that an analyze is in flight is determined. I assume it's something that lives of the EState or similar, but I'm not seeing anything obvious.
