Re: Add support for logging the current role

Поиск
Список
Период
Сортировка
От Robert Haas
Тема Re: Add support for logging the current role
Дата
Msg-id AANLkTin58nBcf=fyjKV2XG2vgG8gJpi+qN16z-23vtAf@mail.gmail.com
обсуждение исходный текст
Ответ на Re: Add support for logging the current role  (Stephen Frost <sfrost@snowman.net>)
Ответы Re: Add support for logging the current role  (Stephen Frost <sfrost@snowman.net>)
Список pgsql-hackers
On Wed, Jan 12, 2011 at 10:43 AM, Stephen Frost <sfrost@snowman.net> wrote:
> * Tom Lane (tgl@sss.pgh.pa.us) wrote:
>> +1 for choosing something more mnemonic than "%o", btw.
>
> Alright, not to be *too* ridiculous about this, but I'm feeling like
> '%R' might be better than '%U', if we don't mind overloading a single
> letter based on case.  I've always been annoyed at the lack of
> distinction between 'user' and 'role' in our docs and feel it does lead
> to some confusion.
>
> Updated patch attached, if people agree.  Compiles, passes regressions,
> works as advertised, etc.

I was thinking that %u/%U would have the advantage of implying some
connection between the two things which is in fact present.  %r/%R
seems not quite as good to me.  Also, let's paint it tangerine.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


В списке pgsql-hackers по дате отправления:

Предыдущее
От: Stephen Frost
Дата:
Сообщение: Re: Add support for logging the current role
Следующее
От: "Kevin Grittner"
Дата:
Сообщение: Re: Compatibility GUC for serializable