Revision 30f912a41378bd0ce363276b2e6b9f590520af1d authored by Andrew Gierth on 28 January 2021, 10:53:10 UTC, committed by Andrew Gierth on 28 January 2021, 11:09:49 UTC
When building aggregate expression steps, strict checks need a bailout
jump for when a null value is encountered, so there is a list of steps
that require later adjustment. Adding entries to that list for steps
that aren't actually strict would be harmless, except that there is an
Assert which catches them. This leads to spurious errors on asserts
builds, for data sets that trigger parallel aggregation of an
aggregate with a non-strict deserialization function (no such
aggregates exist in the core system).

Repair by not adding the adjustment entry when it's not needed.

Backpatch back to 11 where the code was introduced.

Per a report from Darafei (Komzpa) of the PostGIS project; analysis
and patch by me.

Discussion: https://postgr.es/m/87mty7peb3.fsf@news-spur.riddles.org.uk
1 parent d43e235
History
File Mode Size
config
contrib
doc
src
.dir-locals.el -rw-r--r-- 730 bytes
.gitattributes -rw-r--r-- 1.5 KB
.gitignore -rw-r--r-- 504 bytes
COPYRIGHT -rw-r--r-- 1.2 KB
GNUmakefile.in -rw-r--r-- 3.9 KB
HISTORY -rw-r--r-- 284 bytes
Makefile -rw-r--r-- 1.6 KB
README -rw-r--r-- 1.2 KB
README.git -rw-r--r-- 728 bytes
aclocal.m4 -rw-r--r-- 490 bytes
configure -rwxr-xr-x 561.2 KB
configure.in -rw-r--r-- 82.2 KB

README

back to top