]> granicus.if.org Git - postgresql/commit
Cache hash index's metapage in rel->rd_amcache.
authorRobert Haas <rhaas@postgresql.org>
Tue, 7 Feb 2017 17:24:25 +0000 (12:24 -0500)
committerRobert Haas <rhaas@postgresql.org>
Tue, 7 Feb 2017 17:35:45 +0000 (12:35 -0500)
commit293e24e507838733aba4748b514536af2d39d7f2
treece34b71b8a538474577535367d01a7e98884facc
parent39c3ca5161911cebafbcce160ef89e5a4faaf569
Cache hash index's metapage in rel->rd_amcache.

This avoids a very significant amount of buffer manager traffic and
contention when scanning hash indexes, because it's no longer
necessary to lock and pin the metapage for every scan.  We do need
some way of figuring out when the cache is too stale to use any more,
so that when we lock the primary bucket page to which the cached
metapage points us, we can tell whether a split has occurred since we
cached the metapage data.  To do that, we use the hash_prevblkno field
in the primary bucket page, which would otherwise always be set to
InvalidBuffer.

This patch contains code so that it will continue working (although
less efficiently) with hash indexes built before this change, but
perhaps we should consider bumping the hash version and ripping out
the compatibility code.  That decision can be made later, though.

Mithun Cy, reviewed by Jesper Pedersen, Amit Kapila, and by me.
Before committing, I made a number of cosmetic changes to the last
posted version of the patch, adjusted _hash_getcachedmetap to be more
careful about order of operation, and made some necessary updates to
the pageinspect documentation and regression tests.
contrib/pageinspect/expected/hash.out
doc/src/sgml/pageinspect.sgml
src/backend/access/hash/README
src/backend/access/hash/hash.c
src/backend/access/hash/hashinsert.c
src/backend/access/hash/hashpage.c
src/backend/access/hash/hashsearch.c
src/include/access/hash.h