> On Sep 11, 2026, at 09:44, Fujii Masao <[email protected]> wrote:
>
> On Wed, Sep 9, 2026 at 12:51 PM Fujii Masao <[email protected]> wrote:
>> One comment: isn't it be better to add a test for exception handling
>> in a deferred constraint trigger at COMMIT? For example, in triggers.sql:
>
> I've added this test. Attached is the updated version of the patch.
>
> BTW, I found that Patrick Reynolds <[email protected]> is listed in the
> From field of the patch. Does that mean he is the author of the patch,
> rather than you? I'm just asking because I need to know who to list in the
> Author tag of the commit message.
>
> Regards,
>
> --
> Fujii Masao
> <v2-0001-Fix-failing-assert-in-deferred-constraint-trigger.patch>
V2 LGTM.
One nitpick is that, the fix covers both COMMIT and PREPARE TRANSACTION, but v2
only adds a regression test for the COMMIT case. Should we test both paths?
Best regards,
--
Chao Li (Evan)
HighGo Software Co., Ltd.
https://www.highgo.com/
