On Thu, Nov 4, 2021 at 2:21 PM houzj.f...@fujitsu.com <houzj.f...@fujitsu.com> wrote: > > 3) > > + oldctx = > MemoryContextSwitchTo(CacheMemoryContext); > + rfnode = > stringToNode(TextDatumGetCString(rfdatum)); > + exprstate = > pgoutput_row_filter_init_expr(rfnode); > + entry->exprstates = > lappend(entry->exprstates, exprstate); > + MemoryContextSwitchTo(oldctx); > + } > > Currently in the patch, it save and execute each expression separately. I was > thinking it might be better if we can use "AND" to combine all the expressions > into one expression, then we can initialize and optimize the final expression > and execute it only once. >
Fixed in v38-0003 [1]. ------ [1] https://www.postgresql.org/message-id/CAHut%2BPvWCS%2BW_OLV60AZJucY1RFpkXS%3DhfvYWwpwyMvifdJxiQ%40mail.gmail.com Kind Regards, Peter Smith. Fujitsu Australia
