diff options
| author | Teodor Sigaev <teodor@sigaev.ru> | 2008-08-23 10:40:03 +0000 |
|---|---|---|
| committer | Teodor Sigaev <teodor@sigaev.ru> | 2008-08-23 10:40:03 +0000 |
| commit | 0ab43c96781f37a72ae4e1e897d1e10f56c484d2 (patch) | |
| tree | e03337ef4a768b60e2af2ba97f1bdc464a259116 /src/backend/access/gist/gistscan.c | |
| parent | 3fcb7f0bd46b279da20ac3a9ffffda898c4cf573 (diff) | |
| download | postgresql-0ab43c96781f37a72ae4e1e897d1e10f56c484d2.tar.gz postgresql-0ab43c96781f37a72ae4e1e897d1e10f56c484d2.zip | |
Fix possible duplicate tuples while GiST scan. Now page is processed
at once and ItemPointers are collected in memory.
Remove tuple's killing by killtuple() if tuple was moved to another
page - it could produce unaceptable overhead.
Backpatch up to 8.1 because the bug was introduced by GiST's concurrency support.
Diffstat (limited to 'src/backend/access/gist/gistscan.c')
| -rw-r--r-- | src/backend/access/gist/gistscan.c | 14 |
1 files changed, 13 insertions, 1 deletions
diff --git a/src/backend/access/gist/gistscan.c b/src/backend/access/gist/gistscan.c index 187e3bec832..88e1c6ecbee 100644 --- a/src/backend/access/gist/gistscan.c +++ b/src/backend/access/gist/gistscan.c @@ -8,7 +8,7 @@ * Portions Copyright (c) 1994, Regents of the University of California * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/access/gist/gistscan.c,v 1.68 2008/01/01 19:45:46 momjian Exp $ + * $PostgreSQL: pgsql/src/backend/access/gist/gistscan.c,v 1.68.2.1 2008/08/23 10:40:03 teodor Exp $ * *------------------------------------------------------------------------- */ @@ -61,6 +61,7 @@ gistrescan(PG_FUNCTION_ARGS) ReleaseBuffer(so->markbuf); so->markbuf = InvalidBuffer; } + } else { @@ -81,6 +82,7 @@ gistrescan(PG_FUNCTION_ARGS) */ ItemPointerSetInvalid(&so->curpos); ItemPointerSetInvalid(&so->markpos); + so->nPageData = so->curPageData = 0; /* Update scan key, if a new one is given */ if (key && scan->numberOfKeys > 0) @@ -148,6 +150,11 @@ gistmarkpos(PG_FUNCTION_ARGS) so->markbuf = so->curbuf; } + so->markNPageData = so->nPageData; + so->markCurPageData = so->curPageData; + if ( so->markNPageData > 0 ) + memcpy( so->markPageData, so->pageData, sizeof(ItemPointerData) * so->markNPageData ); + PG_RETURN_VOID(); } @@ -197,6 +204,11 @@ gistrestrpos(PG_FUNCTION_ARGS) so->curbuf = so->markbuf; } + so->nPageData = so->markNPageData; + so->curPageData = so->markNPageData; + if ( so->markNPageData > 0 ) + memcpy( so->pageData, so->markPageData, sizeof(ItemPointerData) * so->markNPageData ); + PG_RETURN_VOID(); } |