diff options
| author | David Rowley <drowley@postgresql.org> | 2024-01-22 22:45:33 +1300 |
|---|---|---|
| committer | David Rowley <drowley@postgresql.org> | 2024-01-22 22:45:33 +1300 |
| commit | 74f770ef202fd3f1f1d1f0889f04c16440d23bd0 (patch) | |
| tree | fee1c931b8ef1a3bd96c8d0c70dfdf29cdbe470a /src/backend/utils/adt/xml.c | |
| parent | 5b5318c387451e3eb89eddb4574e57a61297102f (diff) | |
| download | postgresql-74f770ef202fd3f1f1d1f0889f04c16440d23bd0.tar.gz postgresql-74f770ef202fd3f1f1d1f0889f04c16440d23bd0.zip | |
Re-disallow Memoize for parameterized nested loops with join filters
This was previously fixed in 9e215378d but got broken again as a result
of 2489d76c4. It seems that commit causes ppi_clauses to contain
duplicate clauses and it's no longer safe to check the list_length of
that list to determine if there are join conditions other than what's
mentioned in ppi_clauses.
Here we adjust the check to count the distinct rinfo_serial mentioned in
ppi_clauses. We expect that extra->restrictlist won't have duplicate
rinfo_serials.
Reported-by: Amadeo Gallardo
Author: Richard Guo
Discussion: https://postgr.es/m/CADFREbW-BLJd7-a5J%2B5wjVumeFG1ByXiSOFzMtkmY_SDWckTxw%40mail.gmail.com
Backpatch-through: 16, where 2489d76c4 was introduced.
Diffstat (limited to 'src/backend/utils/adt/xml.c')
0 files changed, 0 insertions, 0 deletions