On Sat, Aug 13, 2016 at 4:36 AM, Amit Kapila <amit.kapil...@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 -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers