On Mon, Aug 1, 2016 at 8:45 PM, Tom Lane wrote:
> Amit Kapila writes:
>> On Mon, Aug 1, 2016 at 1:58 AM, Tom Lane wrote:
>>> I believe this is wrong and the CHECK_FOR_INTERRUPTS needs to be before
>>> or after the two latch
Amit Kapila writes:
> On Mon, Aug 1, 2016 at 1:58 AM, Tom Lane wrote:
>> I believe this is wrong and the CHECK_FOR_INTERRUPTS needs to be before
>> or after the two latch operations. As-is, if the reason somebody set
>> our latch was to get us to
On Mon, Aug 1, 2016 at 1:58 AM, Tom Lane wrote:
> Both shm_mq.c and nodeGather.c contain instances of this coding pattern:
>
> WaitLatch(MyLatch, WL_LATCH_SET, 0);
> CHECK_FOR_INTERRUPTS();
> ResetLatch(MyLatch);
>
> I believe this is wrong
Both shm_mq.c and nodeGather.c contain instances of this coding pattern:
WaitLatch(MyLatch, WL_LATCH_SET, 0);
CHECK_FOR_INTERRUPTS();
ResetLatch(MyLatch);
I believe this is wrong and the CHECK_FOR_INTERRUPTS needs to be before
or after the two latch