Re: logical replication seems broken - Mailing list pgsql-hackers

From Amit Kapila
Subject Re: logical replication seems broken
Date
Msg-id CAA4eK1KOTYUTze9QhvakDymQvSBQT8Nz9pJzv+Q8erhQ6XmNEg@mail.gmail.com
Whole thread Raw
In response to Re: logical replication seems broken  (vignesh C <vignesh21@gmail.com>)
Responses Re: logical replication seems broken
Re: logical replication seems broken
List pgsql-hackers
On Mon, Feb 15, 2021 at 11:53 AM vignesh C <vignesh21@gmail.com> wrote:
>
> On Sat, Feb 13, 2021 at 5:58 PM Erik Rijkers <er@xs4all.nl> wrote:
> >
> >
> > I compiled just now a binary from HEAD, and a binary from HEAD+patch
> >
> > HEAD is still broken; your patch rescues it, so yes, fixed.
> >
> > Maybe a test (check or check-world) should be added to run a second replica?  (Assuming that would have caught this
bug)
> >
>
> +1 for the idea of having a test for this. I have written a test for this.
> Thanks for the fix Amit, I could reproduce the issue without your fix
> and verified that the issue gets fixed with the patch you shared.
> Attached a patch for the same. Thoughts?
>

I have slightly modified the comments in the test case to make things
clear. I am planning not to backpatch this because there is no way in
the core code to hit this prior to commit ce0fdbfe97 and we haven't
received any complaints so far. What do you think?

-- 
With Regards,
Amit Kapila.

Attachment

pgsql-hackers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: Refactoring HMAC in the core code
Next
From: Fujii Masao
Date:
Subject: Re: ERROR: invalid spinlock number: 0