]> git.ipfire.org Git - thirdparty/postgresql.git/commit
Fix handling of NULLs when merging BRIN summaries
authorTomas Vondra <tomas.vondra@postgresql.org>
Thu, 18 May 2023 11:00:31 +0000 (13:00 +0200)
committerTomas Vondra <tomas.vondra@postgresql.org>
Thu, 18 May 2023 21:33:45 +0000 (23:33 +0200)
commit80f64b90088a8aeae6aeace2d344c8116ec28061
tree01b3b4f4c44c3658e2775c8ce6967bf3fc16b711
parenteaf99e4c4ae4084b8261e5e1f9c2c93004a98638
Fix handling of NULLs when merging BRIN summaries

When merging BRIN summaries, union_tuples() did not correctly update the
target hasnulls/allnulls flags. When merging all-NULL summary into a
summary without any NULL values, the result had both flags set to false
(instead of having hasnulls=true).

This happened because the code only considered the hasnulls flags,
ignoring the possibility the source summary has allnulls=true.

Discovered while investigating issues with handling empty BRIN ranges
and handling of NULL values, but it's a separate problem (has nothing to
do with empty ranges).

Fixed by considering both flags on the source summary, and updating the
hasnulls flag on the target summary.

Backpatch to 11. The bug exists since 9.5 (where BRIN indexes were
introduced), but those releases are EOL already.

Discussion: https://postgr.es/m/9d993d0d-e431-2196-9ccc-0554d0e60154%40enterprisedb.com
src/backend/access/brin/brin.c