Cache by-reference missing values in a long lived context
authorAndrew Dunstan <andrew@dunslane.net>
Tue, 22 Aug 2023 15:57:08 +0000 (11:57 -0400)
committerAndrew Dunstan <andrew@dunslane.net>
Tue, 22 Aug 2023 19:15:45 +0000 (15:15 -0400)
commit7f4515a58ebd0ae967638b0a27c730cbd49b3455
tree820d0f40b6466017cc0a6ac43e81dbdce18a59f0
parent5a32af3f2c6106080da72e78a42c8fc3ab0584a1
Cache by-reference missing values in a long lived context

Attribute missing values might be needed past the lifetime of the tuple
descriptors from which they are extracted. To avoid possibly using
pointers for by-reference values which might thus be left dangling, we
cache a datumCopy'd version of the datum in the TopMemoryContext. Since
we first search for the value this only needs to be done once per
session for any such value.

Original complaint from Tom Lane, idea for mitigation by Andrew Dunstan,
tweaked by Tom Lane.

Backpatch to version 11 where missing values were introduced.

Discussion: https://postgr.es/m/1306569.1687978174@sss.pgh.pa.us
src/backend/access/common/heaptuple.c
src/tools/pgindent/typedefs.list