diff --git a/src/backend/optimizer/path/costsize.c b/src/backend/optimizer/path/costsize.c index c30d6e84672..89ca4e08bf1 100644 --- a/src/backend/optimizer/path/costsize.c +++ b/src/backend/optimizer/path/costsize.c @@ -2589,11 +2589,6 @@ cost_material(Path *path, double nbytes = relation_byte_size(tuples, width); double work_mem_bytes = work_mem * (Size) 1024; - if (path->parallel_workers == 0 && - path->parent != NULL && - (path->parent->pgs_mask & PGS_CONSIDER_NONPARTIAL) == 0) - enabled = false; - path->rows = tuples; /* diff --git a/src/backend/optimizer/path/joinpath.c b/src/backend/optimizer/path/joinpath.c index 1e4246b49d5..e0c00e26dd5 100644 --- a/src/backend/optimizer/path/joinpath.c +++ b/src/backend/optimizer/path/joinpath.c @@ -1895,8 +1895,17 @@ match_unsorted_outer(PlannerInfo *root, /* * Consider materializing the cheapest inner path, unless that is * disabled or the path in question materializes its output anyway. + * + * At present, we only consider materialization for non-partial outer + * paths, so it's correct to test PGS_CONSIDER_NONPARTIAL here. If we + * ever want to consider materialization for partial paths, we'll need + * to create matpath whenever PGS_NESTLOOP_MATERIALIZE is set, use it + * for partial paths either way, and use it for non-partial paths only + * when PGS_CONSIDER_NONPARTIAL is also set. */ - if ((extra->pgs_mask & PGS_NESTLOOP_MATERIALIZE) != 0 && + if ((extra->pgs_mask & + (PGS_NESTLOOP_MATERIALIZE | PGS_CONSIDER_NONPARTIAL)) == + (PGS_NESTLOOP_MATERIALIZE | PGS_CONSIDER_NONPARTIAL) && inner_cheapest_total != NULL && !ExecMaterializesOutput(inner_cheapest_total->pathtype)) matpath = (Path *) diff --git a/src/backend/optimizer/plan/createplan.c b/src/backend/optimizer/plan/createplan.c index e5200f4b3ce..a50260290fa 100644 --- a/src/backend/optimizer/plan/createplan.c +++ b/src/backend/optimizer/plan/createplan.c @@ -6526,10 +6526,6 @@ materialize_finished_plan(Plan *subplan) subplan->startup_cost -= initplan_cost; subplan->total_cost -= initplan_cost; - /* Clear fields that cost_material() will consult */ - matpath.parallel_workers = 0; - matpath.parent = NULL; - /* Set cost data */ cost_material(&matpath, enable_material,