diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2005-05-15 21:19:55 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2005-05-15 21:19:55 +0000 |
| commit | 2ef172a2a4d8c093c0f2d4d580d5aab1ca8dcffe (patch) | |
| tree | 44d64e7d8c43ed64d37622f90a949c5bb918f0c3 /src/backend/access | |
| parent | eb0d00a9becb865c51c4e4c43f522d5c20addfe3 (diff) | |
| download | postgresql-2ef172a2a4d8c093c0f2d4d580d5aab1ca8dcffe.tar.gz postgresql-2ef172a2a4d8c093c0f2d4d580d5aab1ca8dcffe.zip | |
Fix latent bug in ExecSeqRestrPos: it leaves the plan node's result slot
in an inconsistent state. (This is only latent because in reality
ExecSeqRestrPos is dead code at the moment ... but someday maybe it won't
be.) Add some comments about what the API for plan node mark/restore
actually is, because it's not immediately obvious.
Diffstat (limited to 'src/backend/access')
| -rw-r--r-- | src/backend/access/index/indexam.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/src/backend/access/index/indexam.c b/src/backend/access/index/indexam.c index e2f8078b235..13b04a06b17 100644 --- a/src/backend/access/index/indexam.c +++ b/src/backend/access/index/indexam.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/access/index/indexam.c,v 1.80 2005/04/14 20:03:23 tgl Exp $ + * $PostgreSQL: pgsql/src/backend/access/index/indexam.c,v 1.81 2005/05/15 21:19:54 tgl Exp $ * * INTERFACE ROUTINES * index_open - open an index relation by relation OID @@ -411,6 +411,10 @@ index_markpos(IndexScanDesc scan) /* ---------------- * index_restrpos - restore a scan position + * + * NOTE: this only restores the internal scan state of the index AM. + * The current result tuple (scan->xs_ctup) doesn't change. See comments + * for ExecRestrPos(). * ---------------- */ void |