Re: Use of ActiveSnapshot

On 5/14/2007 4:26 PM, Tom Lane wrote:
> Jan Wieck <JanWieck@Yahoo.com> writes:
>> Which means that the 8.3 fix for the reproducible backend crash, I 
>> posted earlier, is to have SPI_cursor_open() save and restore 
>> ActiveSnapshot while calling RevalidateCachedPlan(). I'll cross check 
>> that this fixes this symptom and commit later today.
> 
> No, the correct fix is to do that inside RevalidateCachedPlan ... and I
> already did it.

Works for me. It fixed the Slony test that actually tripped over the 
bug. Thanks.


Jan

-- 
#======================================================================#
# It's easier to get forgiveness for being wrong than for being right. #
# Let's break this rule - forgive me.                                  #
#================================================== JanWieck@Yahoo.com #