Commit 35313832 authored by Tom Lane's avatar Tom Lane

Just noticed that you can't Query-Cancel a long planner run, because

no part of the planner did CHECK_FOR_INTERRUPTS().  Add one in a
suitably strategic spot.
parent 1bdec342
......@@ -8,7 +8,7 @@
*
*
* IDENTIFICATION
* $PostgreSQL: pgsql/src/backend/optimizer/util/pathnode.c,v 1.120 2005/04/25 01:30:13 tgl Exp $
* $PostgreSQL: pgsql/src/backend/optimizer/util/pathnode.c,v 1.121 2005/06/03 19:00:12 tgl Exp $
*
*-------------------------------------------------------------------------
*/
......@@ -277,6 +277,12 @@ add_path(RelOptInfo *parent_rel, Path *new_path)
ListCell *p1_prev = NULL;
ListCell *p1;
/*
* This is a convenient place to check for query cancel --- no part
* of the planner goes very long without calling add_path().
*/
CHECK_FOR_INTERRUPTS();
/*
* Loop to check proposed new path against old paths. Note it is
* possible for more than one old path to be tossed out because
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment