| From: | Corey Huinker <corey(dot)huinker(at)gmail(dot)com> |
|---|---|
| To: | Tomas Vondra <tomas(at)vondra(dot)me> |
| Cc: | pgsql-hackers(at)lists(dot)postgresql(dot)org |
| Subject: | Re: Extended Statistics set/restore/clear functions. |
| Date: | 2025-01-24 00:06:48 |
| Message-ID: | CADkLM=egTkirK39BUxpDvUakWuojv3bYxosS8bKceNNqFAvBWA@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
>
> I see there's a couple MCV-specific functions in the extended_stats.c.
>> Shouldn't those go into mvc.c instead?
>>
>
> I wanted to put it there, but there was a reason I didn't and I've now
> forgotten what it was. I'll make an effort to relocate it to mcv.c.
>
Looking at it now, I see that check_mcvlist_array() expects access to
extarginfo, so I either I add a bunch of Datum-aware and extarginfo-aware
code to mcv.c, or I do those checks outside of import_mcvlist() and leave
check_mcvlist_array() where it is, which is my current inclination, though
obviously it's not a perfectly clean break.
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Peter Smith | 2025-01-24 00:24:24 | Re: Log a warning in pg_createsubscriber for max_slot_wal_keep_size |
| Previous Message | Paul Jungwirth | 2025-01-23 23:54:15 | Re: SQL:2011 application time |