| From: | Michael Paquier <michael(at)paquier(dot)xyz> |
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | Corey Huinker <corey(dot)huinker(at)gmail(dot)com>, jian he <jian(dot)universality(at)gmail(dot)com>, Tomas Vondra <tomas(at)vondra(dot)me>, pgsql-hackers(at)lists(dot)postgresql(dot)org |
| Subject: | Re: Extended Statistics set/restore/clear functions. |
| Date: | 2025-12-05 03:39:05 |
| Message-ID: | aTJT2arh0B6dWLci@paquier.xyz |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Thu, Dec 04, 2025 at 09:29:50PM -0500, Tom Lane wrote:
> Ah, right, now that you mention it, 56b1e88c8 did this same fix
> before. This way is fine with me.
Oh, I haven't noticed this one. Pushed the bits to take care of the
compiler warnings, for now.
> I didn't review the error message changes closely. I do wonder if
> some of them are can't-happen cases that could just use an elog()
> instead of a translatable message.
Some of them are, yes. However, they are also worded with the same
format as some of the legit cases. So they don't add any extra
workload on the translation side as far as I recall, and I've been
fond of the errdetail part to get a consistent style across the board.
I'll double-check the whole a bit later, attached is the rest of them.
Corey, any comments about these?
--
Michael
| Attachment | Content-Type | Size |
|---|---|---|
| v2-0001-Improve-pg_dependencies.c-and-pg_ndistinct.c.patch | text/x-diff | 74.9 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | vignesh C | 2025-12-05 03:54:18 | Re: Proposal: Conflict log history table for Logical Replication |
| Previous Message | Tom Lane | 2025-12-05 02:29:50 | Re: Extended Statistics set/restore/clear functions. |