On Tue, May 25, 2021 at 6:36 PM Ajin Cherian <itsajin@gmail.com> wrote:
>
> On Tue, Apr 27, 2021 at 1:49 PM Ajin Cherian <itsajin@gmail.com> wrote:
>
> Rebased the patch as it was no longer applying.
Thanks for the updated patch, few comments:
1) I'm not sure if we could add some tests for skip empty
transactions, if possible add a few tests.
2) We could add some debug level log messages for the transaction that
will be skipped.
3) You could keep this variable below the other bool variables in the structure:
+ bool sent_begin_txn; /* flag indicating whether begin
+
* has already been sent */
+
4) You can split the comments to multi-line as it exceeds 80 chars
+ /* output BEGIN if we haven't yet, avoid for streaming and
non-transactional messages */
+ if (!data->sent_begin_txn && !in_streaming && transactional)
+ pgoutput_begin(ctx, txn);
Regards,
Vignesh
