On 26 February 2015 at 05:43, Stephen Frost <sfr...@snowman.net> wrote: > Dean, > > * Dean Rasheed (dean.a.rash...@gmail.com) wrote: >> Attached is a patch to make RLS checks run before attempting to >> insert/update any data rather than afterwards. > > Excellent, this I really like and it's a pretty straight-forward change. > I wonder if there are some documentation updates which need to be done > for this also? I'm planning to look as I vauguely recall mentioning the > ordering of operations somewhere along the way. > > I also addressed the bitrot from the column-priv leak patch. Would be > great to have you take a look at the latest and let me know if you have > any further comments or suggestions. I'm definitely looking forward to > getting these changes in soon. >
Thanks for the update. I don't recall any mention of ordering in the docs, but if there isn't anything there it makes sense to add something. I haven't thought about this patch in a while, but I still like the look of it. It's definitely neater to do these checks earlier and it's good to be able to get RLS-specific errors too. I'll take a look at the latest updates. Regards, Dean -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers