On 02/12/2020 20:13, Heikki Linnakangas wrote:
> On 01/12/2020 16:52, Pavel Borisov wrote:
>> Status update for a commitfest entry.
>>
>> The patch is Waiting on Author for some time. As this is a bug fix,
>> I am
>> moving it to the next CF.
>> Ian, are you planning to continue working on it?
>>
>> As a reviewer, I consider the patch useful and good overall. The comments I left were purely cosmetic. It's a pity
tome that this bugfix delayed for such a small reason and outdated, therefore. It would be nice to complete this fix on
thenext CF.
>
> Yeah, we really should fix this..
>
> On 16/11/2020 04:49, Ian Lawrence Barwick wrote:
>>> Also, I think Heikki's notion could be fulfilled.
>>
>> I spent a bit of time looking at that suggestion but couldn't actually
>> verify it was an issue which needed fixing.
>
> Attached are two patches. The first patch is your original patch, unmodified
> (except for a cosmetic rename of the test file). The second patch builds on
> that, demonstrating and fixing the issue I mentioned. It took me a while to
> create a repro for it, it's easily masked by incidental full-page writes or
> because rows created by XIDs that are not marked as committed on the other
> timeline are invisible, but succeeded at last.
Aha, many thanks. I wasn't entirely sure what I was looking for there and
recently haven't had the time or energy to dig any further.
Regards
Ian Barwick
--
Ian Barwick https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services