Re: Role Attribute Bitmask Catalog Representation

Adam Brightwell wrote:
> Alvaro and Stephen,
> 
> I propose this patch on top of Adam's v5.  Also included is a full patch
> > against master.
> >
> 
> I have attached an updated patch for review
> (role-attribute-bitmask-v7.patch).
> 
> This patch incorporates the 'v5a' patch proposed by Alvaro, input
> validation (Assert) check in 'check_role_attribute' and the documentation
> updates requested by Stephen.

Pushed with a couple of small changes (Catalog.pm complained about the
lack of a CATALOG() line in the new acldefs.h file; you had
pg_role_all_attributes as returning "bool" in the table, but it returns
text[]; and I added index entries for the new functions.)

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