On Thu, Jul 22, 2021 at 11:37 PM Ajin Cherian <itsajin@gmail.com> wrote:
>
I have some minor comments on the v9 patch:
(1) Several whitespace warnings on patch application
(2) Suggested patch comment change:
BEFORE:
The current logical replication behaviour is to send every transaction to
subscriber even though the transaction is empty (because it does not
AFTER:
The current logical replication behaviour is to send every transaction to
subscriber even though the transaction might be empty (because it does not
(3) Comment needed for added struct defn:
typedef struct PGOutputTxnData
(4) Improve comment.
Can you add a comma (or add words) in the below sentence, so we know
how to read it?
+ /*
+ * Delegate to assign the begin sent flag as false same as for the
+ * BEGIN message.
+ */
Regards,
Greg Nancarrow
Fujitsu Australia