On 2013-10-07 15:02:36 -0400, Robert Haas wrote:
> On Fri, Oct 4, 2013 at 3:20 PM, Andres Freund <andres@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.
Sounds good to me. Maybe add a comment to the added regression test
explaining it tests invalidation processing at xact abort?
Greetings,
Andres Freund
-- Andres Freund http://www.2ndQuadrant.com/PostgreSQL Development, 24x7 Support, Training &
Services