You can not select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.

Tom Lane e08d74ca13 Allow plan nodes with initPlans to be considered parallel-safe. ... If the plan itself is parallel-safe, and the initPlans are too, there's no reason anymore to prevent the plan from being marked parallel-safe. That restriction (dating to commit ab77a5a45) was really a special case of the fact that we couldn't transmit subplans to parallel workers at all. We fixed that in commit 5e6d8d2bb and follow-ons, but this case never got addressed. We still forbid attaching initPlans to a Gather node that's inserted pursuant to debug_parallel_query = regress. That's because, when we hide the Gather from EXPLAIN output, we'd hide the initPlans too, causing cosmetic regression diffs. It seems inadvisable to kluge EXPLAIN to the extent required to make the output look the same, so just don't do it in that case. Along the way, this also takes care of some sloppiness about updating path costs to match when we move initplans from one place to another during createplan.c and setrefs.c. Since all the planning decisions are already made by that point, this is just cosmetic; but it seems good to keep EXPLAIN output consistent with where the initplans are. The diff in query_planner() might be worth remarking on. I found that one because after fixing things to allow parallel-safe initplans, one partition_prune test case changed plans (as shown in the patch) --- but only when debug_parallel_query was active. The reason proved to be that we only bothered to mark Result nodes as potentially parallel-safe when debug_parallel_query is on. This neglects the fact that parallel-safety may be of interest for a sub-query even though the Result itself doesn't parallelize. Discussion: https://postgr.es/m/1129530.1681317832@sss.pgh.pa.us		3 years ago
..
backend	Allow plan nodes with initPlans to be considered parallel-safe.	3 years ago
bin	Remove wal_sync_method=fsync_writethrough on Windows.	3 years ago
common	Generate automatically code and documentation related to wait events	3 years ago
fe_utils	Handle \v as a whitespace character in parsers	3 years ago
include	Allow plan nodes with initPlans to be considered parallel-safe.	3 years ago
interfaces	Remove --disable-thread-safety and related code.	3 years ago
makefiles	Remove --disable-thread-safety and related code.	3 years ago
pl	Translation updates	3 years ago
port	Teach in-tree getopt_long() to move non-options to the end of argv.	3 years ago
template	…
test	Allow plan nodes with initPlans to be considered parallel-safe.	3 years ago
timezone	Pre-beta mechanical code beautification.	3 years ago
tools	pg_bsd_indent: Collect test diffs in test.diffs instead of tests.diff	3 years ago
tutorial	Pre-beta mechanical code beautification.	3 years ago
.gitignore	…
DEVELOPERS	…
Makefile	…
Makefile.global.in	Remove --disable-thread-safety and related code.	3 years ago
Makefile.shlib	…
meson.build	…
nls-global.mk	…