projects / postgresql.git / commitdiff
summary | shortlog | log | commit | commitdiff | tree
raw | patch | inline | side by side (parent: b75fedc)
Fix obsolete nbtree array advancement comment.
authorPeter Geoghegan <pg@bowt.ie>
Mon, 28 Apr 2025 16:49:17 +0000 (12:49 -0400)
committerPeter Geoghegan <pg@bowt.ie>
Mon, 28 Apr 2025 16:49:17 +0000 (12:49 -0400)
Checking if another primitive scan is required after all once the next
leaf page was moved from _bt_checkkeys to its _bt_readpage caller by
commit 9a2e2a28.  Update a comment that incorrectly described the
recheck mechanism as something that takes place in _bt_checkkeys.

Also fix an older typo in related code comments.

src/backend/access/nbtree/nbtutils.c patch | blob | blame | history

diff --git a/src/backend/access/nbtree/nbtutils.c b/src/backend/access/nbtree/nbtutils.c
index c899168c99563e9c5d4889214ad18918a29d1b78..11802a4c21518e79b605c8909cbf1036a06a4342 100644 (file)
--- a/src/backend/access/nbtree/nbtutils.c
+++ b/src/backend/access/nbtree/nbtutils.c
@@ -1110,7 +1110,7 @@ _bt_rewind_nonrequired_arrays(IndexScanDesc scan, ScanDirection dir)
  *
  * readpagetup callers must only call here when _bt_check_compare already set
  * continuescan=false.  We help these callers deal with _bt_check_compare's
- * inability to distinguishing between the < and > cases (it uses equality
+ * inability to distinguish between the < and > cases (it uses equality
  * operator scan keys, whereas we use 3-way ORDER procs).  These callers pass
  * a _bt_check_compare-set sktrig value that indicates which scan key
  * triggered the call (!readpagetup callers just pass us sktrig=0 instead).
@@ -1950,7 +1950,7 @@ _bt_advance_array_keys(IndexScanDesc scan, BTReadPageState *pstate,
     * keys for one or more truncated attribute values (scan keys required in
     * _either_ scan direction).
     *
-    * There is a chance that _bt_checkkeys (which checks so->scanBehind) will
+    * There is a chance that _bt_readpage (which checks so->scanBehind) will
     * find that even the sibling leaf page's finaltup is < the new array
     * keys.  When that happens, our optimistic policy will have incurred a
     * single extra leaf page access that could have been avoided.
This is the main PostgreSQL git repository.