On Sat, May 23, 2026 at 10:40 AM Japin Li <[email protected]> wrote:
> Thanks for updating the patch. LGTM. Just one nitpick.
>
> +       int                     pid;                    /* backend pid (0 for 
> slots) */
>
> Per the code, the prepared transaction is also associated with a PID of zero.

Thanks for the review! You're right, prepared transactions also have
pid 0. Fixed the comment to mention that.

Additionally, I forgot to update meson.build, which caused the tests
to fail. I have fixed that in the attached patch.

-- 
Best regards,
Shinya Kato
NTT OSS Center

Attachment: v4-0001-Report-oldest-xmin-blocker-when-VACUUM-cannot-rem.patch
Description: Binary data

Reply via email to