Umm, just skimming here -- this patch shows some LOG messages using elog() rather than ereport(), which seems bogus to me.
Also: "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 context. 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 https://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services -- Sent via pgsql-hackers mailing list (email@example.com) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers