diff options
author | Tom Lane <tgl@sss.pgh.pa.us> | 2011-04-19 18:51:08 -0400 |
---|---|---|
committer | Tom Lane <tgl@sss.pgh.pa.us> | 2011-04-19 18:51:08 -0400 |
commit | 5a4305fdf7ecf8218051f4f2a151d57fa71b9e6b (patch) | |
tree | 8841f188f0a4a4e6fac6f85da8cb16ebad175cf4 /src/backend/access/gist/gistvacuum.c | |
parent | 1f0ad40656fc0cca935cd20038064e6281a55d30 (diff) | |
download | postgresql-5a4305fdf7ecf8218051f4f2a151d57fa71b9e6b.tar.gz postgresql-5a4305fdf7ecf8218051f4f2a151d57fa71b9e6b.zip |
Avoid changing an index's indcheckxmin horizon during REINDEX.
There can never be a need to push the indcheckxmin horizon forward, since
any HOT chains that are actually broken with respect to the index must
pre-date its original creation. So we can just avoid changing pg_index
altogether during a REINDEX operation.
This offers a cleaner solution than my previous patch for the problem
found a few days ago that we mustn't try to update pg_index while we are
reindexing it. System catalog indexes will always be created with
indcheckxmin = false during initdb, and with this modified code we should
never try to change their pg_index entries. This avoids special-casing
system catalogs as the former patch did, and should provide a performance
benefit for many cases where REINDEX formerly caused an index to be
considered unusable for a short time.
Back-patch to 8.3 to cover all versions containing HOT. Note that this
patch changes the API for index_build(), but I believe it is unlikely that
any add-on code is calling that directly.
Diffstat (limited to 'src/backend/access/gist/gistvacuum.c')
0 files changed, 0 insertions, 0 deletions