| 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:33 |
| Message-ID: | E1xMcOz-00000001Kz4-35Ip@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_17_STABLE
Details
-------
https://git.postgresql.org/pg/commitdiff/1fd6ed1f8cf5e0e40d2c66ea0ac4788af757b937
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:43 | pgsql: Remove unwarranted Assert in dependencies_clauselist_selectivity |
| Previous Message | David Rowley | 2026-10-30 02:31:22 | pgsql: Remove unwarranted Assert in dependencies_clauselist_selectivity |