Re: [PATCH v2] use has_privs_for_role for predefined roles - Mailing list pgsql-hackers

From Robert Haas
Subject Re: [PATCH v2] use has_privs_for_role for predefined roles
Date
Msg-id CA+Tgmoa9G20=4=wZ3ai8GrrJhYhoGbqqCsJkKJbv8YpLth5VCg@mail.gmail.com
Whole thread Raw
In response to Re: [PATCH v2] use has_privs_for_role for predefined roles  (Nathan Bossart <nathandbossart@gmail.com>)
Responses Re: [PATCH v2] use has_privs_for_role for predefined roles  (Joshua Brindle <joshua.brindle@crunchydata.com>)
List pgsql-hackers
On Wed, Feb 9, 2022 at 1:13 PM Nathan Bossart <nathandbossart@gmail.com> wrote:
> I do wonder if users find the differences between predefined roles and role
> attributes confusing.  INHERIT doesn't govern role attributes, but it will
> govern predefined roles when this patch is applied.  Maybe the role
> attribute system should eventually be deprecated in favor of using
> predefined roles for everything.  Or perhaps the predefined roles should be
> converted to role attributes.

I couldn't agree more. Apparently it's even confusing to developers,
because otherwise (1) we wouldn't have the problem the patch proposes
to fix in the first place and (2) I would have immediately been
convinced of the value of the patch once it showed up. Since those
things didn't happen, this is apparently confusing to (1) whoever
wrote the code that this patch fixes and (2) me.

-- 
Robert Haas
EDB: http://www.enterprisedb.com



pgsql-hackers by date:

Previous
From: Magnus Hagander
Date:
Subject: Re: is the base backup protocol used by out-of-core tools?
Next
From: Magnus Hagander
Date:
Subject: Re: New developer papercut - Makefile references INSTALL