On Sat, Aug 13, 2016 at 4:36 AM, Amit Kapila <amit.kapila16@gmail.com> wrote:
> AFAICS, your patch seems to be the right fix for this issue, unless we
> need the instrumentation information during execution (other than for
> explain) for some purpose.
Hmm, I disagree. It should be the job of
ExecParallelRetrieveInstrumentation to allocate its data in the
correct context, not the responsibility of nodeGather.c to work around
the fact that it doesn't. The worker instrumentation should be
allocated in the same context as the regular instrumentation
information, which I assume is probably the per-query context.
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company