Hi Fujii-san,

On Mon, Oct 19, 2026 at 8:55 AM Fujii Masao <[email protected]> wrote:
>
> On Sat, Oct 17, 2026 at 11:32 PM Pierre Forstmann
> <[email protected]> wrote:
> >
> > The following review has been posted through the commitfest application:
> > make installcheck-world:  tested, passed
> > Implements feature:       tested, passed
> > Spec compliant:           not tested
> > Documentation:            not tested
> >
> > Hello,
> >
> > This patch looks good to me.
>
> +1. I have one small suggestion, though.
>
> The existing TAP test 009_twophase.pl already tests server startup with
> prepared transactions. So, isn't it better to add the log checks there,
> as in the attached patch?
>

Thanks for the review and suggestions. I agree that we can verify
these changes by adding log checks to the existing TAP test for
two-phase transactions. I’ve done that in the attached patch, which is
based on the changes suggested in your earlier patch. Please take a
look and let me know if you have any other comments.

--
With Regards,
Ashutosh Sharma.

Attachment: v2-0001-Reduce-logging-during-prepared-transaction-recovery.patch
Description: Binary data

Reply via email to