Re: Additional role attributes && superuser review

From: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>
To: Stephen Frost <sfrost(at)snowman(dot)net>
Cc: Adam Brightwell <adam(dot)brightwell(at)crunchydatasolutions(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, Andrew Dunstan <andrew(at)dunslane(dot)net>, Petr Jelinek <petr(at)2ndquadrant(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Additional role attributes && superuser review
Date: 2015-03-01 14:36:59
Message-ID: 20150301143659.GO2384@alvh.no-ip.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Stephen Frost wrote:

> 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.

I thought I saw a comment about using underscore to separate words in
privilege names, such as EXCLUSIVE_BACKUP rather than running it all
together. Was that idea discarded?

--
Álvaro Herrera http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Alvaro Herrera 2015-03-01 14:37:50 Re: rm static libraries before rebuilding
Previous Message Kevin Grittner 2015-03-01 14:08:29 Re: Reduce pinning in btree indexes