On Tue, Jan 24, 2023 at 5:02 AM Euler Taveira <eu...@eulerto.com> wrote: > > > - elog(DEBUG2, "sending feedback (force %d) to recv %X/%X, write %X/%X, > flush %X/%X in-delayed: %d", > + elog(DEBUG2, "sending feedback (force %d) to recv %X/%X, write %X/%X, > flush %X/%X, apply delay: %s", > force, > LSN_FORMAT_ARGS(recvpos), > LSN_FORMAT_ARGS(writepos), > LSN_FORMAT_ARGS(flushpos), > - in_delayed_apply); > + in_delayed_apply? "yes" : "no"); > > It is better to use a string to represent the yes/no option. >
I think it is better to be consistent with the existing force parameter which is also boolean, otherwise, it will look odd. -- With Regards, Amit Kapila.
