"Simon Riggs" <simon(at)2ndquadrant(dot)com> writes:
>> After looking over the code it seems that the executor needs a limited
>> subset of the Query fields, namely
>> ...
>> which I think we should put into a new TopPlan node type.
> All else sounds good, but why would we be caching a plan that used these
> fields?
Um, what's your point? I certainly have no desire to support two
different Executor APIs depending on whether we think the command might
be worth cacheing or not.
regards, tom lane