Andrew Gierth <andrew@tao11.riddles.org.uk> writes:
> "Andrew" == Andrew Gierth <andrew@tao11.riddles.org.uk> writes:
> Andrew> I'm guessing that locally saving/restoring the scan direction
> Andrew> in ExecSubPlan is going to be the best option; it seems to fix
> Andrew> the problem, I'll post a patch in a bit.
> It turns out to be also necessary to do this in ExecSetParamPlan, though
> I couldn't find a way to make a stable regression test for that - my
> manual tests were based on putting a subselect inside a volatile
> construct like CASE WHEN random() < x THEN.
Looks sane to me ... and a bit astonishing that we didn't run into
this a decade or two back.
regards, tom lane