Re: BUG #14758: Segfault with logical replication on a function index

On Thu, Aug 3, 2017 at 12:36 AM, Peter Eisentraut
<peter(dot)eisentraut(at)2ndquadrant(dot)com> wrote:
> On 8/1/17 21:23, Masahiko Sawada wrote:
>> On Wed, Aug 2, 2017 at 7:52 AM, Andres Freund <andres(at)anarazel(dot)de> wrote:
>>> On 2017-08-01 16:28:31 -0400, Peter Eisentraut wrote:
>>>> On 7/28/17 06:07, Masahiko Sawada wrote:
>>>>> I could reproduced this issue and it seems to me that the cause of
>>>>> this is that the table sync worker didn't get a snapshot before
>>>>> starting table copy. Attached patch fixes this problem.
>>>>
>>>> Should it be GetTransactionSnapshot() rather than GetLatestSnapshot()?
>>>
>>> Don't think it makes a meaningful difference here, but I agree it's more
>>> consistent.
>>
>> +1.
>
> committed with that change
>

Thank you!

Regards,

--
Masahiko Sawada
NIPPON TELEGRAPH AND TELEPHONE CORPORATION
NTT Open Source Software Center