On Tue, Apr 5, 2016 at 11:47 PM, Robert Haas <robertmh...@gmail.com> wrote: > On Mon, Apr 4, 2016 at 4:28 AM, Fujii Masao <masao.fu...@gmail.com> wrote: >>>> + ereport(LOG, >>>> + (errmsg("standby \"%s\" is now the synchronous standby with priority %u", >>>> + application_name, MyWalSnd->sync_standby_priority))); >>>> >>>> s/ the / a / >> >> I have no objection to this change itself. But we have used this message >> in 9.5 or before, so if we apply this change, probably we need >> back-patching. > > "the" implies that there can be only one synchronous standby at that > priority, while "a" implies that there could be more than one. So the > situation might be different with this patch than previously. (I > haven't read the patch so I don't know whether this is actually true, > but it might be what Thomas was going for.)
Thanks for the explanation! I applied that change, to the latest patch I posted upthread. Regards, -- Fujii Masao -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers