> 3)
> + ereport(LOG,
> + (errmsg("reset synced flag for replication slot \"%s\"",
> + NameStr(s->data.name))));
>
> a) Shall we change it to DEBUG1?
> b) Shall the msg be:
> synced flag reset for replication slot \"%s\" during promotion
>I think this can stay as a LOG message, it only runs once at startup and applies just to logical slots, so it won’t be noisy. I’d also avoid mentioning “during promotion,” since the flag might accidentally be set on the primary and then reset later during startup, making that description inaccurate. -- With Regards, Ashutosh Sharma.
