Thread: pgsql: Ignore BRIN indexes when checking for HOT udpates

pgsql: Ignore BRIN indexes when checking for HOT udpates

From
Tomas Vondra
Date:
Ignore BRIN indexes when checking for HOT udpates

When determining whether an index update may be skipped by using HOT, we
can ignore attributes indexed only by BRIN indexes. There are no index
pointers to individual tuples in BRIN, and the page range summary will
be updated anyway as it relies on visibility info.

This also removes rd_indexattr list, and replaces it with rd_attrsvalid
flag. The list was not used anywhere, and a simple flag is sufficient.

Patch by Josef Simanek, various fixes and improvements by me.

Author: Josef Simanek
Reviewed-by: Tomas Vondra, Alvaro Herrera
Discussion: https://postgr.es/m/CAFp7QwpMRGcDAQumN7onN9HjrJ3u4X3ZRXdGFT0K5G2JWvnbWg%40mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/5753d4ee320b3f6fb2ff734667a1ce1d9d8615a1

Modified Files
--------------
doc/src/sgml/indexam.sgml                        | 11 +++
src/backend/access/brin/brin.c                   |  1 +
src/backend/access/gin/ginutil.c                 |  1 +
src/backend/access/gist/gist.c                   |  1 +
src/backend/access/hash/hash.c                   |  1 +
src/backend/access/heap/heapam.c                 |  2 +-
src/backend/access/nbtree/nbtree.c               |  1 +
src/backend/access/spgist/spgutils.c             |  1 +
src/backend/utils/cache/relcache.c               | 50 ++++++++------
src/include/access/amapi.h                       |  2 +
src/include/utils/rel.h                          |  3 +-
src/include/utils/relcache.h                     |  4 +-
src/test/modules/dummy_index_am/dummy_index_am.c |  1 +
src/test/regress/expected/brin.out               | 85 ++++++++++++++++++++++++
src/test/regress/sql/brin.sql                    | 63 ++++++++++++++++++
15 files changed, 202 insertions(+), 25 deletions(-)