On 2/28/15 10:10 PM, Stephen Frost wrote:
> Adam,
>
> * Adam Brightwell (adam.brightwell@crunchydatasolutions.com) wrote:
>> I have attached and updated patch for review.
>
> Thanks! I've gone over this and made quite a few documentation and
> comment updates, but not too much else, so I'm pretty happy with how
> this is coming along. As mentioned elsewhere, this conflicts with the
> GetUserId() to has_privs_of_role() cleanup, but as I anticipate handling
> both this patch and that one, I'll find some way to manage. :)
>
> Updated patch attached. Barring objections, I'll be moving forward with
> this soonish. Would certainly appreciate any additional testing or
> review that you (or anyone!) has time to provide.
Let's move this discussion to the right thread.
Why are we not using roles and function execute privileges for this?