Re: Additional role attributes && superuser review

Hi,

On 2015/12/23 7:23, Stephen Frost wrote:
> Updated patch attached.  I'll give it another good look and then commit
> it, barring objections.

Just a minor nitpick about a code comment -
    /*
+     * Check that the user is not trying to create a role in the reserved
+     * "pg_" namespace.
+     */
+    if (IsReservedName(stmt->role))

The wording may be slightly confusing, especially saying "... in ...
namespace". ISTM, "namespace" is fairly extensively used around the code
to mean something like "a schema's namespace".

Could perhaps be reworded as:
    /*
+     * Check that the user is not trying to create a role with reserved
+     * name, ie, one starting with "pg_".

If OK, there seems to be one more place further down in the patch with
similar wording.

Thanks,
Amit