Add vacuum_delay_point call in compute_index_stats's per-sample-row loop.

tglsfdc · tglsfdc · commit f444de5e34f6 · 2015-03-29T15:04:18.000-04:00
Slow functions in index expressions might cause this loop to take long
enough to make it worth being cancellable.  Probably it would be enough
to call CHECK_FOR_INTERRUPTS here, but for consistency with other
per-sample-row loops in this file, let's use vacuum_delay_point.

Report and patch by Jeff Janes.  Back-patch to all supported branches.
diff --git a/src/backend/commands/analyze.c b/src/backend/commands/analyze.c
@@ -743,6 +743,8 @@ compute_index_stats(Relation onerel, double totalrows,
 		{
 			HeapTuple	heapTuple = rows[rowno];
 
+			vacuum_delay_point();
+
 			/*
 			 * Reset the per-tuple context each time, to reclaim any cruft
 			 * left behind by evaluating the predicate or index expressions.

Original file line number	Diff line number	Diff line change
`@@ -743,6 +743,8 @@ compute_index_stats(Relation onerel, double totalrows,`
`743`	`743`	`{`
`744`	`744`	`HeapTuple heapTuple = rows[rowno];`
`745`	`745`
	`746`	`+ vacuum_delay_point();`
	`747`	`+`
`746`	`748`	`/*`
`747`	`749`	`* Reset the per-tuple context each time, to reclaim any cruft`
`748`	`750`	`* left behind by evaluating the predicate or index expressions.`