On Tue, Nov 28, 2017 at 7:13 PM, Robert Haas <robertmhaas@xxxxxxxxx> wrote:
On Tue, Nov 28, 2017 at 2:32 AM, Dilip Kumar <dilipbalaut@xxxxxxxxx> wrote:
> I think BitmapHeapScan check whether dsa is valid or not if DSA is not
> valid then it should assume it's non-parallel plan.
>
> Attached patch should fix the issue.
So, create the pstate and then pretend we didn't? Why not just avoid
creating it in the first place, like this?
This is better way to fix it.
I haven't checked whether this fixes the bug, but if it does, we can
avoid introducing an extra branch in BitmapHeapNext.
With my test it's fixing the problem.