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