| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | Andres Freund <andres(at)anarazel(dot)de> |
| Cc: | Andrew Dunstan <andrew(at)dunslane(dot)net>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: generating function default settings from pg_proc.dat |
| Date: | 2026-02-17 18:41:49 |
| Message-ID: | 1881128.1771353709@sss.pgh.pa.us |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Here's the promised v2, which addresses all your review comments.
With respect to the list of supported types in bki.sgml: I wonder if
we should just drop that, because it evidently hasn't been maintained
well. It wasn't at all in sync with the actual contents of TypInfo[].
I made it be so, but ...
Poking further at that, I found that there were a lot of TypInfo[]
entries that were not actually used and seem to have just been
cargo-culted in. So this patch removes all the ones that aren't
demonstrably necessary to get through initdb. Maybe that's too
aggressive, but in view of the potential for maintenance errors
(cf 7cdb633c8) I don't think we should be carrying unused entries
there.
regards, tom lane
| Attachment | Content-Type | Size |
|---|---|---|
| v2-0001-Simplify-creation-of-built-in-functions-with-defa.patch | text/x-diff | 41.2 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Noah Misch | 2026-02-17 18:47:58 | EUC_* encodings: pass check-world |
| Previous Message | Peter Geoghegan | 2026-02-17 18:17:15 | Re: index prefetching |