On 1/3/17 1:26 AM, amul sul wrote: > One more requirement for pg_background is session, command_qh, > response_qh and worker_handle should be last longer than current > memory context, for that we might need to allocate these in > TopMemoryContext. Please find attach patch does the same change in > BackgroundSessionStart().
I had pondered this issue extensively. The standard coding convention in postgres is that the caller sets the memory context. See the dblink and plpython patches that make this happen in their own way. I agree it would make sense that you either pass in a memory context or always use TopMemoryContext. I'm open to these ideas, but they did not seem to match any existing usage. -- Peter Eisentraut http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
