• Robert Haas's avatar
    Set consider_parallel correctly for upper planner rels. · 5ce5e4a1
    Robert Haas authored
    Commit 3fc6e2d7 introduced new "upper"
    RelOptInfo structures but didn't set consider_parallel for them
    correctly, a point I completely missed when reviewing it.  Later,
    commit e06a3896 made the situation
    worse by doing it incorrectly for the grouping relation.  Try to
    straighten all of that out.  Along the way, get rid of the annoying
    wholePlanParallelSafe flag, which was only necessarily because of
    the fact that upper planning stages didn't use paths at the time
    that code was written.
    
    The most important immediate impact of these changes is that
    force_parallel_mode will provide useful test coverage in quite a few
    more scenarios than it did previously, but it's also necessary
    preparation for fixing some problems related to subqueries.
    
    Patch by me, reviewed by Tom Lane.
    5ce5e4a1
planner.c 161 KB