mirror of
https://github.com/postgres/postgres.git
synced 2025-06-26 12:21:12 +03:00
Support unlogged GiST index.
The reason this wasn't supported before was that GiST indexes need an increasing sequence to detect concurrent page-splits. In a regular WAL- logged GiST index, the LSN of the page-split record is used for that purpose, and in a temporary index, we can get away with a backend-local counter. Neither of those methods works for an unlogged relation. To provide such an increasing sequence of numbers, create a "fake LSN" counter that is saved and restored across shutdowns. On recovery, unlogged relations are blown away, so the counter doesn't need to survive that either. Jeevan Chalke, based on discussions with Robert Haas, Tom Lane and me.
This commit is contained in:
@ -158,16 +158,6 @@ gistbuild(PG_FUNCTION_ARGS)
|
||||
elog(ERROR, "index \"%s\" already contains data",
|
||||
RelationGetRelationName(index));
|
||||
|
||||
/*
|
||||
* We can't yet handle unlogged GiST indexes, because we depend on LSNs.
|
||||
* This is duplicative of an error in gistbuildempty, but we want to check
|
||||
* here so as to throw error before doing all the index-build work.
|
||||
*/
|
||||
if (heap->rd_rel->relpersistence == RELPERSISTENCE_UNLOGGED)
|
||||
ereport(ERROR,
|
||||
(errcode(ERRCODE_FEATURE_NOT_SUPPORTED),
|
||||
errmsg("unlogged GiST indexes are not supported")));
|
||||
|
||||
/* no locking is needed */
|
||||
buildstate.giststate = initGISTstate(index);
|
||||
|
||||
@ -204,7 +194,7 @@ gistbuild(PG_FUNCTION_ARGS)
|
||||
PageSetTLI(page, ThisTimeLineID);
|
||||
}
|
||||
else
|
||||
PageSetLSN(page, GetXLogRecPtrForTemp());
|
||||
PageSetLSN(page, gistGetFakeLSN(heap));
|
||||
|
||||
UnlockReleaseBuffer(buffer);
|
||||
|
||||
|
Reference in New Issue
Block a user