Hi, Thanks for addressing my previous comments. Patch v14-0001 looks good to me, except I have one question:
The patch uses errmsg_internal() for the logging, but I noticed the only other code using GUC 'log_replication_commands' has errmsg() instead of errmsg_internal(). Isn't it better to be consistent with the existing code? ====== Kind Regards, Peter Smith. Fujitsu Australia