| 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:31:43 |
| Message-ID: | E1xMcP9-00000001Kzk-2f0w@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
------
REL_16_STABLE
Details
-------
https://git.postgresql.org/pg/commitdiff/9ffd2018aa77b6d01d4ef81279ea3f9cc429bc9f
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:53 | pgsql: Remove unwarranted Assert in dependencies_clauselist_selectivity |
| Previous Message | David Rowley | 2026-10-30 02:31:33 | pgsql: Remove unwarranted Assert in dependencies_clauselist_selectivity |