| From: | Michael Paquier <michael(at)paquier(dot)xyz> |
|---|---|
| To: | Jim Jones <jim(dot)jones(at)uni-muenster(dot)de> |
| Cc: | surya poondla <suryapoondla4(at)gmail(dot)com>, Kirill Reshke <reshkekirill(at)gmail(dot)com>, Postgres hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Zsolt Parragi <zsolt(dot)parragi(at)percona(dot)com> |
| Subject: | Re: Reject access to unlogged tables in contrib modules |
| Date: | 2026-11-09 08:10:23 |
| Message-ID: | avF_7yZ9qjCYW4DC@paquier.xyz |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Fri, Nov 06, 2026 at 07:59:14PM +0100, Jim Jones wrote:
> 0001 (pageinspect) and 0002 (pgstattuple) add the helper function
> check_relation_accessible(), where the tests for unlogged and temporary
> tables now live. While at it, I moved the existing
> RELATION_IS_OTHER_TEMP(rel) check to the same function, so that all
> checks are kept in one place.
v5-0001 and v5-0002 have both been applied with a couple of tweaks. I
was not feeling strongly about the need for the TOAST check in
pageinspect. I suspect that it does not matter with the check for the
main relation still around, and that's less syscache lookups to worry
about.
--
Michael
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Chao Li | 2026-11-09 08:42:01 | Re: extension_control_path: ignore empty elements and deduplicate directories |
| Previous Message | Richard Guo | 2026-11-09 08:10:17 | Re: A gap in preventing bogus parameterizations in a798660ebe3 |