Subject: Re: Fix a typo of func DecodeInsert()

From: Richard Guo <[email-masked]>
Date: Wed, 17 Jan 2024 09:10:33 +0800

On Wed, Jan 17, 2024 at 8:47 AM Yongtao Huang <[email-masked]>
wrote:

> Hi all,
> I think the comment above the function DecodeInsert()
> in src/backend/replication/logical/decode.c should be
> + * *Inserts *can contain the new tuple.
> , rather than
> - * *Deletes *can contain the new tuple.
>

Nice catch.  +1.

I kind of wonder if it would be clearer to state that "XLOG_HEAP_INSERT
can contain the new tuple", in order to differentiate it from
XLOG_HEAP2_MULTI_INSERT.

Thanks
Richard