diff options
| author | David Rowley <drowley@postgresql.org> | 2025-06-14 17:18:31 +1200 |
|---|---|---|
| committer | David Rowley <drowley@postgresql.org> | 2025-06-14 17:18:31 +1200 |
| commit | 2f98f967fa78fd36279989ecdd5fbf74ab332fa9 (patch) | |
| tree | a4c76da4f931fd9df5c5b0d8dfb9bd8a4f2cfaf6 /src/backend/executor | |
| parent | 0fe50417eca46d6993da350845dc09d7e899c6d6 (diff) | |
| download | postgresql-2f98f967fa78fd36279989ecdd5fbf74ab332fa9.tar.gz postgresql-2f98f967fa78fd36279989ecdd5fbf74ab332fa9.zip | |
Improve comments for TidRangeEval
Here we provide a bit more detail on why TidRangeEval() does return false
when trss_mintid is greater than trss_maxtid.
Reported-by: Junwang Zhao <zhjwpku@gmail.com>
Author: David Rowley <dgrowleyml@gmail.com>
Reviewed-by: Junwang Zhao <zhjwpku@gmail.com>
Discussion: https://postgr.es/m/CAEG8a3KUbUUqQgfK5X8Sj-%2BppPtGNTU%2BZiep0Rxr7SLjoR%2BB6w%40mail.gmail.com
Diffstat (limited to 'src/backend/executor')
| -rw-r--r-- | src/backend/executor/nodeTidrangescan.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/backend/executor/nodeTidrangescan.c b/src/backend/executor/nodeTidrangescan.c index ab2eab9596e..26f7420b64b 100644 --- a/src/backend/executor/nodeTidrangescan.c +++ b/src/backend/executor/nodeTidrangescan.c @@ -128,9 +128,11 @@ TidExprListCreate(TidRangeScanState *tidrangestate) * TidRangeEval * * Compute and set node's block and offset range to scan by evaluating - * the trss_tidexprs. Returns false if we detect the range cannot + * node->trss_tidexprs. Returns false if we detect the range cannot * contain any tuples. Returns true if it's possible for the range to - * contain tuples. + * contain tuples. We don't bother validating that trss_mintid is less + * than or equal to trss_maxtid, as the scan_set_tidrange() table AM + * function will handle that. * ---------------------------------------------------------------- */ static bool |