aboutsummaryrefslogtreecommitdiff
path: root/src/backend/optimizer/plan/planner.c
diff options
context:
space:
mode:
Diffstat (limited to 'src/backend/optimizer/plan/planner.c')
-rw-r--r--src/backend/optimizer/plan/planner.c6
1 files changed, 6 insertions, 0 deletions
diff --git a/src/backend/optimizer/plan/planner.c b/src/backend/optimizer/plan/planner.c
index 5da863d85de..17c82b376f4 100644
--- a/src/backend/optimizer/plan/planner.c
+++ b/src/backend/optimizer/plan/planner.c
@@ -327,6 +327,11 @@ standard_planner(Query *parse, const char *query_string, int cursorOptions,
* if we want to allow parallel inserts in general; updates and deletes
* have additional problems especially around combo CIDs.)
*
+ * We don't try to use parallel mode unless interruptible. The leader
+ * expects ProcessInterrupts() calls to reach HandleParallelMessages().
+ * Even if we called HandleParallelMessages() another way, starting a
+ * parallel worker is too delay-prone to be prudent when uncancellable.
+ *
* For now, we don't try to use parallel mode if we're running inside a
* parallel worker. We might eventually be able to relax this
* restriction, but for now it seems best not to have parallel workers
@@ -337,6 +342,7 @@ standard_planner(Query *parse, const char *query_string, int cursorOptions,
parse->commandType == CMD_SELECT &&
!parse->hasModifyingCTE &&
max_parallel_workers_per_gather > 0 &&
+ INTERRUPTS_CAN_BE_PROCESSED() &&
!IsParallelWorker())
{
/* all the cheap tests pass, so scan the query tree */