Clarify nbtree array exhaustion comments.
commit81a25790f1f66319b4f088148b97dbb8badd4907
authorPeter Geoghegan <pg@bowt.ie>
Wed, 30 Oct 2024 17:43:49 +0000 (30 13:43 -0400)
committerPeter Geoghegan <pg@bowt.ie>
Wed, 30 Oct 2024 17:43:49 +0000 (30 13:43 -0400)
tree51ff6817f7dbc11dab4fb69fbcb3d34932820f9c
parent849110dd3eec3e21c358e24f11c6d501d05eee72
Clarify nbtree array exhaustion comments.

Strictly speaking, we only need to make sure to leave the scan's array
keys in their final positions (final for the current scan direction) to
handle SAOP array exhaustion because btgettuple might only return a
subset of the items for the final page (final for the current scan
direction), before the scan changes direction.  While it's typical for
so->currPos to be invalidated shortly after the scan's arrays are first
exhausted, and while so->currPos invalidation does obviate the need to
leave the scan's arrays in any particular state, we can't rely on any of
that actually happening when handling array exhaustion.  Adjust comments
to make all of that a lot clearer.

Oversight in commit 5bf748b8, which enhanced nbtree ScalarArrayOp
execution.
src/backend/access/nbtree/nbtutils.c