From: | vignesh C <vignesh21(at)gmail(dot)com> |
---|---|
To: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Alexander Lakhin <exclusion(at)gmail(dot)com>, Noah Misch <noah(at)leadboat(dot)com>, pgsql-hackers(at)lists(dot)postgresql(dot)org |
Subject: | Re: Why is subscription/t/031_column_list.pl failing so much? |
Date: | 2024-02-07 10:57:10 |
Message-ID: | CALDaNm1hLZW4H4Z61swK6WPW6pcNcjzXqw=6NqG7e-RMtkFaZA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Wed, 7 Feb 2024 at 15:21, Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> wrote:
>
> On Tue, Feb 6, 2024 at 8:21 PM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> >
> > Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> writes:
> > > Yeah, I was worried about that. The other idea I have previously
> > > thought was to change Alter Subscription to Drop+Create Subscription.
> > > That should also help in bringing stability without losing any
> > > functionality.
> >
> > Hm, why would that fix it?
> >
>
> Because for new subscriptions, we will start reading WAL from the
> latest WAL insert pointer on the publisher which will be after the
> point where publication is created.
I was able to reproduce the issue consistently with the changes shared
by Tom Lane at [1].
I have made changes to change ALTER SUBSCRIPTION to DROP+CREATE
SUBSCRIPTION and verified that the test has passed consistently for
>50 runs that I ran. Also the test execution time increased for this
case is very negligibly:
Without patch: 7.991 seconds
With test change patch: 8.121 seconds
The test changes for the same are attached.
[1] - https://www.postgresql.org/message-id/631312.1707251789%40sss.pgh.pa.us
Regards,
Vignesh
Attachment | Content-Type | Size |
---|---|---|
v3-0001-Fix-the-intermittent-Build-farm-failures-in-031_c.patch | text/x-patch | 5.0 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Amit Kapila | 2024-02-07 10:59:07 | Re: Synchronizing slots from primary to standby |
Previous Message | Nazir Bilal Yavuz | 2024-02-07 10:54:26 | Re: Streaming I/O, vectored I/O (WIP) |