On Thu, Aug 12, 2021 at 1:21 PM Greg Nancarrow <gregn4...@gmail.com> wrote:
>
> 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"),
>You have a point but I think in this case it might look a bit odd as we have another field 'commit timestamp' after that which is lowercase. -- With Regards, Amit Kapila.
