Re: Fix a typo of func DecodeInsert()
| От | Yongtao Huang |
|---|---|
| Тема | Re: Fix a typo of func DecodeInsert() |
| Дата | |
| Msg-id | CAOe1Go30dX0GA9Yd5CpU72-zxvupZ1Xe38ALt_sHj7HnGjUyEQ@mail.gmail.com обсуждение исходный текст |
| Ответ на | Re: Fix a typo of func DecodeInsert() (Richard Guo <guofenglinux@gmail.com>) |
| Ответы |
Re: Fix a typo of func DecodeInsert()
|
| Список | pgsql-hackers |
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_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_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)
* 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
В списке pgsql-hackers по дате отправления: