Andrei Lepikhov <lepihov(at)gmail(dot)com> writes:
> Thanks for pushing this!
> But could you change this code a little bit?
> I reported this issue a year ago. At that time, it was triggered by the
> CustomScan node [1]. I haven't found the solution there [2]. Your code
> looks like a good tradeoff, and if you slightly change the code (like in
> the attachment), it allows CustomScan to survive such cases.
This seems like it's making assumptions it shouldn't about what
CustomScan does. If there's an argument for doing this, it should
be added to the adjacent comments.
(Also, what's with the random change in contrib/Makefile?)
regards, tom lane