| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | Ayush Tiwari <ayushtiwari(dot)slg01(at)gmail(dot)com> |
| Cc: | PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>, Álvaro Herrera <alvherre(at)kurilemu(dot)de>, Nathan Bossart <nathandbossart(at)gmail(dot)com> |
| Subject: | Re: Remove the refint contrib module (for v20) |
| Date: | 2026-06-05 22:09:47 |
| Message-ID: | 3788238.1780697387@sss.pgh.pa.us |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Ayush Tiwari <ayushtiwari(dot)slg01(at)gmail(dot)com> writes:
> Thoughts? Does this look like the right approach, or is there a
> reason to keep refint around?
The only reason for refint to exist anymore is as example code,
but I think it's well short of our current standards, so it
really would need more work if we want to keep it. Some things
I'd want to see worked on if we keep it:
* The comments are not great
* Error handling is haphazard, eg inconsistent mix of elog and ereport
* After the cache-ectomy, it's hardly clear why it's still using
SPI_prepare rather than just SPI_execute_with_args
I doubt that anyone really wants to put in that work, so I'm
good with just removing it.
regards, tom lane
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Daniel Gustafsson | 2026-06-05 22:22:38 | Re: Fix OAuth validator docs for error_detail on internal errors |
| Previous Message | Daniel Gustafsson | 2026-06-05 22:05:36 | Re: Remove the refint contrib module (for v20) |