Hi Chao, Shihao, Michael, Thank you for the patches, v2 patch looks good. Attached v3 is v2 with the below one additional change, it applies on its own.
There is one more spot, in UpdateLogicalDecodingStatusEndOfRecovery where I felt the comment could be made more clear. "* ... On the other hand, we need to wait for synchronizing * XLogLogicalInfo even if we've not updated the status above as the * status have been turned on and off during recovery, having running * processes have different status on their local caches." Updated the above in v3 as "* ... On the other hand, we need to wait for XLogLogicalInfo to be * synchronized even if we've not updated the status above, as the * status may have been turned on and off during recovery, leaving * running processes with different values in their local caches." I'm not sure if this reword is worth the churn, happy to drop it if you'd rather keep the original wording. Regards, Surya Poondla
v3-0001-Fix-typos-in-logicalctl.c-comments.patch
Description: Binary data
