Re: Evaluate arguments of correlated SubPlans in the referencing ExprState

Hi,

On 2024-07-19 21:17:12 -0700, Andres Freund wrote:
> On 2024-07-18 16:01:19 -0400, Tom Lane wrote:
> > Alena Rybakina <lena(dot)ribackina(at)yandex(dot)ru> writes:
> > > I fixed it. The code remains the same.
> >
> > I see the cfbot is again complaining that this patch doesn't apply.
> >
> > In hopes of pushing this over the finish line, I fixed up the (minor)
> > patch conflict and also addressed the cosmetic complaints I had
> > upthread [1]. I think the attached v4 is committable. If Andres is
> > too busy, I can push it, but really it's his patch ...
>
> Thanks for the rebase - I'll try to get it pushed in the next few days!

And finally done. No code changes. I did spend some more time evaluating the
resource usage benefits actually do exist (see mail upthread).

Thanks for the reviews, rebasing and the reminders!

Greetings,

Andres