Re: Introduce wait_for_subscription_sync for TAP tests

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>
Cc: Thomas Munro <thomas(dot)munro(at)gmail(dot)com>, Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>, "shiy(dot)fnst(at)fujitsu(dot)com" <shiy(dot)fnst(at)fujitsu(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Introduce wait_for_subscription_sync for TAP tests
Date: 2022-09-09 21:45:43
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com> writes:
> On Fri, Sep 9, 2022 at 11:31 PM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>> Recently a number of buildfarm animals have failed at the same
>> place in src/test/subscription/t/ [1][2][3][4]:
>> # Failed test '2x3000 rows in t'
>> # at t/ line 149.
>> # got: '9000'
>> # expected: '6000'
>> # Looks like you failed 1 test of 7.
>> [09:30:56] t/ ......................
>> This was the last commit to touch that test script. I'm thinking
>> maybe it wasn't adjusted quite correctly? On the other hand, since
>> I can't find any similar failures before the last 48 hours, maybe
>> there is some other more-recent commit to blame. Anyway, something
>> is wrong there.

> It seems that this commit is innocent as it changed only how to wait.

Yeah. I was wondering if it caused us to fail to wait somewhere,
but I concur that's not all that likely.

> It's likely that the commit f6c5edb8abcac04eb3eac6da356e59d399b2bcef
> is relevant.

Noting that the errors have only appeared in the past couple of
days, I'm now suspicious of adb466150b44d1eaf43a2d22f58ff4c545a0ed3f
(Fix recovery_prefetch with low maintenance_io_concurrency).

regards, tom lane

In response to


Browse pgsql-hackers by date

  From Date Subject
Next Message Ranier Vilela 2022-09-09 21:49:32 Re: Avoid overhead with fprintf related functions
Previous Message Ranier Vilela 2022-09-09 21:45:31 Re: Avoid overhead with fprintf related functions