diff options
| author | Amit Kapila <akapila@postgresql.org> | 2018-06-20 07:51:42 +0530 |
|---|---|---|
| committer | Amit Kapila <akapila@postgresql.org> | 2018-06-20 07:51:42 +0530 |
| commit | 403318b71f7058ecbfb65bcc7de1eec96cd35d3f (patch) | |
| tree | 38b647ac33a5163828f1f6b2581d1976e6689822 /src/backend/optimizer | |
| parent | 1c7c317cd9d1e5647454deed11b55dae764c83bf (diff) | |
| download | postgresql-403318b71f7058ecbfb65bcc7de1eec96cd35d3f.tar.gz postgresql-403318b71f7058ecbfb65bcc7de1eec96cd35d3f.zip | |
Don't consider parallel append for parallel unsafe paths.
Commit ab72716778 allowed Parallel Append paths to be generated for a
relation that is not parallel safe. Prevent that from happening.
Initial analysis by Tom Lane.
Reported-by: Rajkumar Raghuwanshi
Author: Amit Kapila and Rajkumar Raghuwanshi
Reviewed-by: Amit Khandekar and Robert Haas
Discussion:https://postgr.es/m/CAKcux6=tPJ6nJ08r__nU_pmLQiC0xY15Fn0HvG1Cprsjdd9s_Q@mail.gmail.com
Diffstat (limited to 'src/backend/optimizer')
| -rw-r--r-- | src/backend/optimizer/path/allpaths.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/src/backend/optimizer/path/allpaths.c b/src/backend/optimizer/path/allpaths.c index 146298828d8..3ada379f8bc 100644 --- a/src/backend/optimizer/path/allpaths.c +++ b/src/backend/optimizer/path/allpaths.c @@ -1383,7 +1383,7 @@ add_paths_to_append_rel(PlannerInfo *root, RelOptInfo *rel, List *pa_partial_subpaths = NIL; List *pa_nonpartial_subpaths = NIL; bool partial_subpaths_valid = true; - bool pa_subpaths_valid = enable_parallel_append; + bool pa_subpaths_valid; List *all_child_pathkeys = NIL; List *all_child_outers = NIL; ListCell *l; @@ -1391,6 +1391,9 @@ add_paths_to_append_rel(PlannerInfo *root, RelOptInfo *rel, bool build_partitioned_rels = false; double partial_rows = -1; + /* If appropriate, consider parallel append */ + pa_subpaths_valid = enable_parallel_append && rel->consider_parallel; + /* * AppendPath generated for partitioned tables must record the RT indexes * of partitioned tables that are direct or indirect children of this |