On Fri, Oct 4, 2013 at 3:20 PM, Andres Freund <and...@2ndquadrant.com> wrote:
> On 2013-10-04 15:15:36 -0400, Robert Haas wrote:
>> Andres, are you (or is anyone) going to try to fix this assertion failure?
>
> I think short term replacing it by IsTransactionOrTransactionBlock() is
> the way to go. Entirely restructuring how cache invalidation in the
> abort path works is not a small task.

Well, if we're going to go that route, how about something like the
attached?  I included the assert-change per se, an explanatory
comment, and the test case that Noah devised to cause the current
assertion to fail.

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

Attachment: weaken-searchcatcache-assert.patch
Description: Binary data

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to