Umm, just skimming here -- this patch shows some LOG messages using
elog() rather than ereport(), which seems bogus to me.

        "logical replication table synchronization worker for subscription 
\"%s\", table \"%s\" has started"
surely there is a more convenient name than "logical replication table
synchronization worker" for this process?  I think just getting rid of
the words "logical replication" there would be sufficient, since we
don't have the concept of "table synchronization worker" in any other

More generally, the overall wording of this message seems a bit off.
How about something along the lines of
  "starting synchronization for table \"%s\" in subscription \"%s\""

Álvaro Herrera      
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

Sent via pgsql-hackers mailing list (
To make changes to your subscription:

Reply via email to