| From: | Andreas Karlsson <andreas(at)proxel(dot)se> |
|---|---|
| To: | Joel Jacobson <joel(at)compiler(dot)org>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Marko Tiikkaja <marko(at)joh(dot)to> |
| Subject: | Re: INSERT ... ON CONFLICT DO SELECT [FOR ...] take 2 |
| Date: | 2024-12-17 10:45:52 |
| Message-ID: | 1ba45f71-f0de-479e-878d-8d71fb572484@proxel.se |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On 12/3/24 11:24 AM, Joel Jacobson wrote:
> I've tested the patch successfully and also looked at the code briefly
> and at first glance think it looks nice and clean.
Thanks for the interest!
Here is an updated version which adds support for RLS. I am not 100%
sure that my choices for RLS are correct since I decided to, similar to
ON CONFLICT DO UPDATE, throw an error if the RLS checks fail rather than
filter the RETURNING tuples using the RLS USING clause. I can see a case
for either and am not familiar enough with RLS to have a good intuition
for which.
Andreas
| Attachment | Content-Type | Size |
|---|---|---|
| v4-0001-Add-support-for-ON-CONFLICT-DO-SELECT-FOR.patch | text/x-patch | 37.5 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Dagfinn Ilmari Mannsåker | 2024-12-17 10:47:27 | Re: pg_combinebackup PITR comparison test fix |
| Previous Message | Andrey Borodin | 2024-12-17 10:04:47 | Re: UUID v7 |