On Thu, Jul 8, 2021 at 10:34 AM Euler Taveira <eu...@eulerto.com> wrote: > > Greg, I like your suggestion and already integrate it (I replaced > ExecAllocTableSlot() with MakeSingleTupleTableSlot() because we don't need the > List).
Yes I agree, I found the same thing, it's not needed. >I'm still working on a new version to integrate all suggestions that you > and Peter did. I have a similar code to Peter's plan cache and I'm working on > merging both ideas together. I'm done for today but I'll continue tomorrow. > I also realised that my 0005 patch wasn't handling RelationSyncEntry invalidation, so I've updated it. For completeness, I'm posting the complete patch set with the updates, so you can look at it and compare with yours, and also it'll keep the cfbot happy until you post your updated patch. Regards, Greg Nancarrow Fujitsu Australia
v17-0001-Row-filter-for-logical-replication.patch
Description: Binary data
v17-0002-PS-tmp-describe-intermediate-test-steps.patch
Description: Binary data
v17-0003-PS-tmp-add-more-comments-for-expected-results.patch
Description: Binary data
v17-0004-PS-POC-Implement-a-plan-cache-for-pgoutput.patch
Description: Binary data
v17-0005-Improve-row-filtering-performance.patch
Description: Binary data
