| From: | Dean Rasheed <dean(dot)a(dot)rasheed(at)gmail(dot)com> | 
|---|---|
| To: | jian he <jian(dot)universality(at)gmail(dot)com> | 
| Cc: | Viktor Holmberg <v(at)viktorh(dot)net>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> | 
| Subject: | Re: Docs and tests for RLS policies applied by command type | 
| Date: | 2025-11-03 11:21:49 | 
| Message-ID: | CAEZATCUsiVwCHnAcy-H90exNcNNz35t8EF7PJtQvVEfsms_ytQ@mail.gmail.com | 
| Views: | Whole Thread | Raw Message | Download mbox | Resend email | 
| Thread: | |
| Lists: | pgsql-hackers | 
On Tue, 28 Oct 2025 at 03:34, jian he <jian(dot)universality(at)gmail(dot)com> wrote:
>
> while reading the doc, this description in UPDATE section:
> """
> Note, however, that unlike a standalone UPDATE command, if the existing row does
> not pass the USING expressions, an error will be thrown (the UPDATE path will
> never be silently avoided).
> ""
>
> I think the above statement also applies to MERGE ... THEN UPDATE.
> Perhaps the table “Policies Applied by Command Type” already conveys this,
> but I’m not sure.
Yeah, reading through the text on that page in more detail, there are
a number of other omissions, or places that aren't quite fully
correct, so I've gone through those and attempted to improve things.
Also, I think it would be better if the table made the distinction
between policy checks that just filter out rows, without throwing an
error, and checks that do cause an error to be thrown.
v4 attached.
Regards,
Dean
| Attachment | Content-Type | Size | 
|---|---|---|
| v4-0001-doc-Improve-description-of-RLS-policies-applied-b.patch | text/x-patch | 17.0 KB | 
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Álvaro Herrera | 2025-11-03 11:21:50 | BRIN autosummarization lacking a snapshot | 
| Previous Message | Peter Eisentraut | 2025-11-03 11:16:13 | Re: Reorganize GUC structs |