Re: Add support for logging the current role

Поиск
Список
Период
Сортировка
От Stephen Frost
Тема Re: Add support for logging the current role
Дата
Msg-id 20110115030531.GX4933@tamriel.snowman.net
обсуждение исходный текст
Ответ на Re: Add support for logging the current role  (Tom Lane <tgl@sss.pgh.pa.us>)
Ответы Re: Add support for logging the current role  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers
* Tom Lane (tgl@sss.pgh.pa.us) wrote:
> In any case, if the GUC representation is a list of field names, I think
> the POLA demands that the system honor the list order.

Agreed.  That puts us back into the question of how to make it
efficient.  My best thought at the moment, which doesn't strike me as
particularly efficient, is to build an array of the columns as enum's
and then have loop through the array and use a switch() on the enum.  At
least it's all integer-based there then and we're not calling strcmp()
for every field or strchr to find the next field, but couldn't we do
better?

> BTW, in case you didn't know, there are some GUCs defined as lists of
> identifiers already (look for GUC_LIST bits).  Be sure to steal code.

No, I didn't..  Excellent.
Thanks!
    Stephen

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

Предыдущее
От: Andrew Dunstan
Дата:
Сообщение: Re: Add support for logging the current role
Следующее
От: Stephen Frost
Дата:
Сообщение: Re: Add support for logging the current role