Re: Failure of subscription tests with topminnow

From: Ajin Cherian <itsajin(at)gmail(dot)com>
To: Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>
Cc: Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>, Michael Paquier <michael(at)paquier(dot)xyz>, Postgres hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Failure of subscription tests with topminnow
Date: 2021-08-27 11:03:17
Message-ID: CAFPTHDZtGTpp4NCKKcr7QEDtD65N1kskBxuYjtwGUZamZOZHoQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Fri, Aug 27, 2021 at 3:29 PM Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> wrote:
>
>
> I think the fix is correct but similar changes are required in
> 022_twophase_cascade.pl as well (search for $oldpid in tests). I am
> not completely sure but I think it is better to make this test change
> in back branches as well to make it stable and reduce such random
> build farm failures.

I have made the changes in 022_twophase_cascade.pl for HEAD as well as
patches for older branches.

regards,
Ajin Cherian
Fujitsu Australia

Attachment Content-Type Size
HEAD-v2-0001-fix-for-tap-test-failure-seen-in-001_rep_changes.patch application/octet-stream 4.0 KB
REL-13-STABLE-v1-0001-fix-for-tap-test-failure-seen-in-001_rep_changes.patch application/octet-stream 2.5 KB
REL-12-STABLE-v1-0001-fix-for-tap-test-failure-seen-in-001_rep_changes.patch application/octet-stream 2.5 KB
REL-10-STABLE-v1-0001-fix-for-tap-test-failure-seen-in-001_rep_changes.patch application/octet-stream 2.5 KB
REL-11-STABLE-v1-0001-fix-for-tap-test-failure-seen-in-001_rep_changes.patch application/octet-stream 2.5 KB
REL-14-STABLE-v1-0001-fix-for-tap-test-failure-seen-in-001_rep_changes.patch application/octet-stream 2.5 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Amit Kapila 2021-08-27 11:27:13 Re: Added schema level support for publication.
Previous Message Masahiko Sawada 2021-08-27 11:03:05 Re: Skipping logical replication transactions on subscriber side