| From: | David Rowley <drowley(at)postgresql(dot)org> |
|---|---|
| To: | pgsql-committers(at)lists(dot)postgresql(dot)org |
| Subject: | pgsql: Remove unwarranted Assert in dependencies_clauselist_selectivity |
| Date: | 2026-10-30 02:30:55 |
| Message-ID: | E1xMcON-00000001Kx3-1RW6@gemulon.postgresql.org |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-committers |
Remove unwarranted Assert in dependencies_clauselist_selectivity()
The loop above from where the Assert is removed handles assigning the same
attnum to any duplicate expressions, so asserting there are no duplicate
attnums just after purposefully making duplicate ones isn't the correct
thing to do.
The Assert was added by a4d75c86bf, so backpatch to v14
Author: BHUVAN CHILUKUTI <c(dot)bhuvan(at)iitg(dot)ac(dot)in>
Reviewed-by: David Rowley <dgrowleyml(at)gmail(dot)com>
Discussion: https://postgr.es/m/PN0P287MB0903563C70D8E5B81912E855A7602@PN0P287MB0903.INDP287.PROD.OUTLOOK.COM
Backpatch-through: 14
Branch
------
master
Details
-------
https://git.postgresql.org/pg/commitdiff/b0392c26bd3cfb61ee24b7577967cc806d01da36
Modified Files
--------------
src/backend/statistics/dependencies.c | 11 -----------
1 file changed, 11 deletions(-)
| From | Date | Subject | |
|---|---|---|---|
| Next Message | David Rowley | 2026-10-30 02:31:08 | pgsql: Remove unwarranted Assert in dependencies_clauselist_selectivity |
| Previous Message | Richard Guo | 2026-10-30 01:51:36 | pgsql: Fix nestloop parameter assignment for PHVs evaluated above outer |