On Fri, Nov 12, 2021 at 9:19 PM Ajin Cherian <itsa...@gmail.com> wrote: > > Attaching version 39- >
Thanks for the updated patch. Some review comments: doc/src/sgml/ref/create_publication.sgml (1) improve comment + /* Set up a pstate to parse with */ "pstate" is the variable name, better to use "ParseState". src/test/subscription/t/025_row_filter.pl (2) rename TAP test 025 to 026 I suggest that the t/025_row_filter.pl TAP test should be renamed to 026 now because 025 is being used by some schema TAP test. (3) whitespace errors The 0006 patch applies with several whitespace errors. (4) fix crash The pgoutput.c patch that I previously posted on this thread needs to be applied to fix the coredump issue reported by Tang-san. While that fixes the crash, I haven't tracked through to see where/whether the expression nodes are actually freed or whether now there is a possible memory leak issue that may need further investigation. Regards, Greg Nancarrow
