From: Alvaro Herrera <alvherre@alvh.no-ip.org>
Date: Fri, 3 Nov 2017 19:36:32 +0000 (+0100)
Subject: Fix thinkos in BRIN summarization
X-Git-Tag: REL_11_BETA1~1290
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=1b890562b8d1b44bd3ef948aeeb58dd59abd04b7;p=thirdparty%2Fpostgresql.git

Fix thinkos in BRIN summarization

The previous commit contained a thinko that made a single-range
summarization request process from there to end of table.  Fix by
setting the correct end range point.  Per buildfarm.
---

diff --git a/src/backend/access/brin/brin.c b/src/backend/access/brin/brin.c
index 8636620f647..cafc8fe7bea 100644
--- a/src/backend/access/brin/brin.c
+++ b/src/backend/access/brin/brin.c
@@ -1292,8 +1292,11 @@ brinsummarize(Relation index, Relation heapRel, BlockNumber pageRange,
 	if (pageRange == BRIN_ALL_BLOCKRANGES)
 		startBlk = 0;
 	else
+	{
 		startBlk = (pageRange / pagesPerRange) * pagesPerRange;
-	if (startBlk >= heapNumBlocks)
+		heapNumBlocks = Min(heapNumBlocks, startBlk + pagesPerRange);
+	}
+	if (startBlk > heapNumBlocks)
 	{
 		/* Nothing to do if start point is beyond end of table */
 		brinRevmapTerminate(revmap);