| From: | Dean Rasheed <dean(dot)a(dot)rasheed(at)gmail(dot)com> |
|---|---|
| To: | Joe Conway <joe(dot)conway(at)crunchydata(dot)com> |
| Cc: | Noah Misch <noah(at)leadboat(dot)com>, Stephen Frost <sfrost(at)snowman(dot)net>, Robert Haas <robertmhaas(at)gmail(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: more RLS oversights |
| Date: | 2015-07-29 16:40:18 |
| Message-ID: | CAEZATCU-U7tcpec+1G0dsUBOeuMqMyMVcZTN9ybuqoZMs4iwPg@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On 29 July 2015 at 16:52, Joe Conway <joe(dot)conway(at)crunchydata(dot)com> wrote:
> On 07/29/2015 02:41 AM, Dean Rasheed wrote:
>> I don't think there is any point in adding the new function
>> transformPolicyClause(), which is identical to transformWhereClause().
>> You can just use transformWhereClause() with EXPR_KIND_POLICY. It's
>> already used for lots of other expression kinds.
>
> Yeah, I was debating that. Although I do think the name
> transformWhereClause() is unfortunate. Maybe it ought to be renamed
> transformBooleanExpr() or something similar?
>
I expect it's probably being used by other people's code though, so
renaming it might cause pain for quite a few people.
Regards,
Dean
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Joe Conway | 2015-07-29 16:52:13 | Re: more RLS oversights |
| Previous Message | Corey Huinker | 2015-07-29 16:40:07 | Re: dblink: add polymorphic functions. |