Re: Additional role attributes && superuser review - Mailing list pgsql-hackers

From Alvaro Herrera
Subject Re: Additional role attributes && superuser review
Date
Msg-id 20150302155907.GA3291@alvh.no-ip.org
Whole thread Raw
In response to Re: Additional role attributes && superuser review  (Adam Brightwell <adam.brightwell@crunchydatasolutions.com>)
Responses Re: Additional role attributes && superuser review  (Stephen Frost <sfrost@snowman.net>)
List pgsql-hackers
Adam Brightwell wrote:
> Alvaro,
> 
> > 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?
> 
> I'm not sure there was an actual discussion on the topic.  Though, at one
> point I had proposed it as one of the forms of this attribute.  Personally,
> I think it is easier to read with the underscore.  But, ultimately, I
> defaulted to no underscore to remain consistent with the other attributes,
> such as CREATEDB and CREATEROLE.

If we were choosing those names nowadays, would we choose CREATEDB at
all in the first place?  I think we'd go for something more verbose,
probably CREATE_DATABASE.  (CREATEROLE is not as old as CREATEDB, but my
bet is that it was modelled after CREATEUSER without considering the
whole readability topic too much.)

Anyway it doesn't seem to me that consistency with lack of separators in
those very old names should be our guiding principle here.

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



pgsql-hackers by date:

Previous
From: Marco Nenciarini
Date:
Subject: Re: File based Incremental backup v8
Next
From: Stephen Frost
Date:
Subject: Re: Additional role attributes && superuser review