On Sun, Mar 8, 2026 at 7:58 PM Sahitya Chandra <[email protected]> wrote: > > While reading the subscription code I noticed one error message that doesn't > follow the project's style: in check_remote_recovery_state() in > subscriptioncmds.c, the errmsg() for the "publisher is in recovery" case ends > with a period. The convention elsewhere (including the similar message a few > lines above in the same file) is no trailing period. > > The attached one-line patch removes the trailing period from that string. No > behavior change. >
LGTM. -- With Regards, Amit Kapila.
