On Thu, Aug 12, 2021 at 3:54 PM Masahiko Sawada <sawada.m...@gmail.com> wrote: > > I've attached the updated patches. FYI I've included the patch > (v8-0005) that fixes the assertion failure during shared fileset > cleanup to make cfbot tests happy. >
A minor comment on the 0001 patch: In the message I think that using "ID" would look better than lowercase "id" and AFAICS it's more consistent with existing messages. + appendStringInfo(&buf, _(" in transaction id %u with commit timestamp %s"), Regards, Greg Nancarrow Fujitsu Australia