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
