On Wed, Feb 14, 2024 at 7:51 PM Euler Taveira <euler@eulerto.com> wrote:
>
> On Wed, Feb 14, 2024, at 8:45 AM, Amit Kapila wrote:
>
> Now, I am less clear about whether to quote "logical" or not in the
> above message. Do you have any suggestions?
>
>
> The possible confusion comes from the fact that the sentence contains "must be"
> in the middle of a comparison expression. For pg_createsubscriber, we are using
>
> publisher requires wal_level >= logical
>
> I suggest to use something like
>
> slot synchronization requires wal_level >= logical
>
This sounds like a better idea but shouldn't we directly use this in
'errmsg' instead of a separate 'errhint'? Also, if change this, then
we should make a similar change for other messages in the same
function.
--
With Regards,
Amit Kapila.