Re: pgsql: Add isolation tests for snapshot behavior in ri_triggers.c

From: Amit Langote <amitlangote09(at)gmail(dot)com>
To: Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>
Cc: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: Re: pgsql: Add isolation tests for snapshot behavior in ri_triggers.c
Date: 2022-04-11 05:54:29
Message-ID: CA+HiwqGwws0xNvW6CMC6F5haeOxFnvntDhTEZjQst24Z7irKXw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Hi Alvaro,

On Fri, Apr 8, 2022 at 3:32 AM Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> wrote:
> Add isolation tests for snapshot behavior in ri_triggers.c
>
> They are to check the behavior of RI_FKey_check() and
> ri_Check_Pk_Match(). A test case whereby RI_FKey_check() queries a
> partitioned PK table under REPEATABLE READ isolation produces wrong
> output due to a bug of the partition-descriptor logic and that is noted
> as such in the comment in the test. A subsequent commit will fix the
> bug and replace the buggy output by the correct one.

Should this one also have been reverted? I wonder if it might be
better to add this suite with the (yet unwritten) patch to fix the bug
mentioned in the above commit message, which will likely be
back-ported.

--
Amit Langote
EDB: http://www.enterprisedb.com

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message Thomas Munro 2022-04-11 06:48:58 Re: pgsql: Add TAP test for archive_cleanup_command and recovery_end_comman
Previous Message Peter Eisentraut 2022-04-11 05:39:53 pgsql: Put new command-line options into alphabetical order in help out