| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | pgsql-committers(at)lists(dot)postgresql(dot)org |
| Subject: | pgsql: Partially undo commit 94da73281. |
| Date: | 2022-08-05 19:57:59 |
| Message-ID: | E1oK3SJ-000TLt-3f@gemulon.postgresql.org |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-committers |
Partially undo commit 94da73281.
On closer inspection, mcv.c isn't as broken for ScalarArrayOpExpr
as I thought. The Var-on-right issue is real enough, but actually
it does cope fine with a NULL array constant --- I was misled by
an XXX comment suggesting it didn't. Undo that part of the code
change, and replace the XXX comment with something less misleading.
Branch
------
master
Details
-------
https://git.postgresql.org/pg/commitdiff/4c81a50e5b6ea0bf2bc9896a6c89d16163598b43
Modified Files
--------------
src/backend/statistics/extended_stats.c | 7 +++----
src/backend/statistics/mcv.c | 25 ++++++++++++++++---------
2 files changed, 19 insertions(+), 13 deletions(-)
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Peter Geoghegan | 2022-08-05 20:10:39 | pgsql: Add missing parenthesis to max item size macro. |
| Previous Message | Tom Lane | 2022-08-05 19:00:18 | pgsql: Fix handling of bare boolean expressions in mcv_get_match_bitmap |