From: | "Drouvot, Bertrand" <bertranddrouvot(dot)pg(at)gmail(dot)com> |
---|---|
To: | Robert Haas <robertmhaas(at)gmail(dot)com>, Andres Freund <andres(at)anarazel(dot)de> |
Cc: | "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: pgsql: TAP test for logical decoding on standby |
Date: | 2023-05-24 11:58:54 |
Message-ID: | f0e25007-0852-ee4f-7093-bfd65e5c343c@gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
Hi,
On 5/23/23 5:15 PM, Robert Haas wrote:
> On Sat, Apr 8, 2023 at 5:26 AM Andres Freund <andres(at)anarazel(dot)de> wrote:
>> TAP test for logical decoding on standby
>
> Small nitpicks:
>
> 1. The test names generated by check_slots_conflicting_status() start
> with a capital letter, while most other test names start with a
> lower-case letter.
>
Yeah, not sure that would deserve a fix for its own but if we address 2.
then let's do 1. too.
> 2. The function is called 7 times, 6 with a true argument and 1 with a
> false argument, but the test name only depends on whether the argument
> is true or false, so we get the same test name 6 times. Maybe there's
> not a reasonable way to do better, I'm not sure, but it's not ideal.
>
I agree that's not ideal (but one could still figure out which one is
failing if any by looking at the perl script).
If we want to "improve" this, what about passing a second argument that
would provide more context in the test name?
Regards,
--
Bertrand Drouvot
PostgreSQL Contributors Team
RDS Open Source Databases
Amazon Web Services: https://aws.amazon.com
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2023-05-24 13:54:59 | pgsql: doc: PG 16 relnotes, merge and move vector items |
Previous Message | Kyotaro Horiguchi | 2023-05-24 04:13:51 | Re: could not extend file "base/5/3501" with FileFallocate(): Interrupted system call |
From | Date | Subject | |
---|---|---|---|
Next Message | Daniel Gustafsson | 2023-05-24 12:04:26 | Re: Docs: Encourage strong server verification with SCRAM |
Previous Message | Richard Guo | 2023-05-24 11:19:16 | Wrong results due to missing quals |