Thread: Fix a typo of func DecodeInsert()

Fix a typo of func DecodeInsert()

From
Yongtao Huang
Date:
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.

Please correct me if I'm wrong, thanks a lot.
Attachment

Re: Fix a typo of func DecodeInsert()

From
Richard Guo
Date:

On Wed, Jan 17, 2024 at 8:47 AM Yongtao Huang <yongtaoh2022@gmail.com> 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

Re: Fix a typo of func DecodeInsert()

From
Yongtao Huang
Date:
Thank you. I prefer to keep the comments of these three functions DecodeInsert(),  DecodeUpdate(), and DecodeDelete() aligned.
```
/*
 * Parse XLOG_HEAP_INSERT (not MULTI_INSERT!) records into tuplebufs.
 *
 * Inserts can contain the new tuple.
 */
static void
DecodeInsert(LogicalDecodingContext *ctx, XLogRecordBuffer *buf)

/*
 * Parse XLOG_HEAP_UPDATE and XLOG_HEAP_HOT_UPDATE, which have the same layout
 * in the record, from wal into proper tuplebufs.
 *
 * Updates can possibly contain a new tuple and the old primary key.
 */
static void
DecodeUpdate(LogicalDecodingContext *ctx, XLogRecordBuffer *buf)

/*
 * Parse XLOG_HEAP_DELETE from wal into proper tuplebufs.
 *
 * Deletes can possibly contain the old primary key.
 */
static void
DecodeDelete(LogicalDecodingContext *ctx, XLogRecordBuffer *buf)

```

Best wishes

Yongtao Huang 


Richard Guo <guofenglinux@gmail.com> 于2024年1月17日周三 09:10写道:

On Wed, Jan 17, 2024 at 8:47 AM Yongtao Huang <yongtaoh2022@gmail.com> 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

Re: Fix a typo of func DecodeInsert()

From
Michael Paquier
Date:
On Wed, Jan 17, 2024 at 12:18:12PM +0800, Yongtao Huang wrote:
> Thank you. I prefer to keep the comments of these three functions
> *DecodeInsert()*,  *DecodeUpdate()*, and *DecodeDelete()* aligned.

Not sure either what we would gain with a more complicated description
in this area knowing that there is also DecodeMultiInsert(), so I have
just fixed the top of DecodeInsert() as you have suggested as it is
clearly wrong.  Thanks.
--
Michael

Attachment