Re: some last patches breaks plan cache - Mailing list pgsql-hackers

From Peter Eisentraut
Subject Re: some last patches breaks plan cache
Date
Msg-id cba0a9fa-6da7-6a55-c0d4-d7aee1614f7b@2ndquadrant.com
Whole thread Raw
In response to Re: some last patches breaks plan cache  (Tomas Vondra <tomas.vondra@2ndquadrant.com>)
Responses Re: some last patches breaks plan cache  (Pavel Stehule <pavel.stehule@gmail.com>)
List pgsql-hackers
On 4/4/18 14:03, Tomas Vondra wrote:
>> If there's really no other way, you could use a PG_TRY block to 
>> ensure that the pointer gets reset on the way out. But I question
>> why we've got a design that requires that in the first place. It's
>> likely to have more problems than this.
> 
> I agree it needs a solution that does not require us to track and
> manually reset pointers on random places. No argument here.

I've committed a fix with PG_TRY.

A more complete solution would be to able to keep the plan independent
of a resowner.  That would require a bit more deep surgery in SPI, it
seems.  I'll take a look if it's doable.

-- 
Peter Eisentraut              http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services


pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: Excessive PostmasterIsAlive calls slow down WAL redo
Next
From: "Tels"
Date:
Subject: Re: Parallel Aggregates for string_agg and array_agg