Robert Haas <robertmh...@gmail.com> writes:
> On Fri, Nov 27, 2015 at 1:33 AM, Etsuro Fujita
>> One option to avoid that
>> is to set the fdw_outerplan in ExecInitForeignScan as in my patch [1], or
>> BeginForeignScan as you proposed. That breaks the equivalence that the Plan
>> tree and the PlanState tree should be mirror images of each other, but I
>> think that that break would be harmless.
> I'm not sure how many times I have to say this, but we are not doing
> that. I will not commit any patch that does that, and I will
> vigorously argue against anyone else committing such a patch either.
And I'll back him up. That's a horrible idea. You're proposing to break
a very fundamental structural property for the convenience of one little
corner of the system.
regards, tom lane
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
