+ /* + * Advance inStart to show that the copy related message has been + * processed. + */ + conn->inStart = conn->inCursor; This change...
+ /* getCopyStart() moves inStart itself */ conn->asyncStatus = PGASYNC_COPY_IN; - break; + continue; ... And this change are risky for a backpatch. And they actually break the existing replication protocol
Can you please explain how will it break replication protocol?
I have done the required handling for Copy Both mode as well in attached
patch similar to what was done for other Copy modes in previous patch.
Check if you still find it as broken for replication?
I have only kept the changes for COPY modes, so that once we settle on
those, I think similar changes could be done for getParamDescriptions()