Re: [PATCH v2] use has_privs_for_role for predefined roles

Поиск
Список
Период
Сортировка
От Robert Haas
Тема Re: [PATCH v2] use has_privs_for_role for predefined roles
Дата
Msg-id CA+Tgmoa9G20=4=wZ3ai8GrrJhYhoGbqqCsJkKJbv8YpLth5VCg@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [PATCH v2] use has_privs_for_role for predefined roles  (Nathan Bossart <nathandbossart@gmail.com>)
Ответы Re: [PATCH v2] use has_privs_for_role for predefined roles  (Joshua Brindle <joshua.brindle@crunchydata.com>)
Список 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 по дате отправления:

Предыдущее
От: Magnus Hagander
Дата:
Сообщение: Re: is the base backup protocol used by out-of-core tools?
Следующее
От: Magnus Hagander
Дата:
Сообщение: Re: New developer papercut - Makefile references INSTALL