Amit Langote <amitlangote09@gmail.com> writes: > On Thu, Oct 17, 2024 at 1:12 PM Tom Lane <tgl@sss.pgh.pa.us> wrote: >> In the short term, I suspect the only workable fix is to undo the >> optimization of having ExecInitExprRec not recurse into both raw_expr >> and formatted_expr.
> This or actually I'm tempted to simply revert the whole thing > (b6e1157e7d3) as an ill-considered refactoring, because I am not able > to convince myself that calling ExecAggPlainTransByVal() twice, via > both raw_expr and formatted_expr, is always safe.
Not following the concern here? As far as nodeAgg is concerned, they'd be two independent aggregates.
In any case, whatever we do in master, you can't "simply revert" b6e1157e7 in released branches. It changed the way JsonValueExpr is represented in stored rules, and you don't get to undo that midstream.
Sorry, I can't fully understand what you said above. What's the stored rule?
And "you don't get to undo that midstream." What is the Scenario that we get to undo?
