v22b-0027-review.patch

text/x-patch

Filename: v22b-0027-review.patch
Type: text/x-patch
Part: 26
Message: Re: BitmapHeapScan streaming read user and prelim refactoring

Patch

Format: format-patch
Series: patch 0027
Subject: review
File+
src/include/nodes/tidbitmap.h 2 0
From 8a4d7ea0bc7d6d0851c7c352d2181e51ec3d1102 Mon Sep 17 00:00:00 2001
From: Tomas Vondra <tv@fuzzy.cz>
Date: Tue, 18 Jun 2024 23:55:48 +0200
Subject: [PATCH v22b 27/29] review

---
 src/include/nodes/tidbitmap.h | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/include/nodes/tidbitmap.h b/src/include/nodes/tidbitmap.h
index 7efb5ac4ca8..76d97ced9f5 100644
--- a/src/include/nodes/tidbitmap.h
+++ b/src/include/nodes/tidbitmap.h
@@ -50,6 +50,8 @@ typedef struct TBMIterateResult
 	 * 8K pages, or 1024 with 32K pages).  So there's not much point in making
 	 * the per-page bitmaps variable size.  We just legislate that the size is
 	 * this:
+	 *
+	 * XXX Doesn't this make it tied to heap too much?
 	 */
 	OffsetNumber offsets[MaxHeapTuplesPerPage];
 } TBMIterateResult;
-- 
2.45.2