Re: Event trigger bugs (was Re: Repeated crashes in GENERATED ... AS IDENTITY tests)

2018-04-20 Thread Tom Lane
I wrote: > Hence, two questions: > * Should EventTriggerTableRewrite do > if (!currentEventTriggerState || > currentEventTriggerState->commandCollectionInhibited) > return; > like most of the other functions, or should it just check for null > currentEventTriggerState? After cl

Re: Event trigger bugs (was Re: Repeated crashes in GENERATED ... AS IDENTITY tests)

2018-04-19 Thread Alvaro Herrera
Tom Lane wrote: > Hence, two questions: > > * Should EventTriggerTableRewrite do > > if (!currentEventTriggerState || > currentEventTriggerState->commandCollectionInhibited) > return; > > like most of the other functions, or should it just check for null > currentEventTrigge

Event trigger bugs (was Re: Repeated crashes in GENERATED ... AS IDENTITY tests)

2018-04-19 Thread Tom Lane
I wrote: > I'm inclined to say that whether or not there's a bug here (and there > well may be, it doesn't seem like a crash is a good thing), this is > bad test design and we need to change it. So my suspicion was aroused by the fact that, unlike almost every other function in event_trigger.c, Ev